Well, it looks like `defaultBranch` will be a `NameError` currently.

In any case, calling everything p4* seems redundant. When one is setting environment variables, one needs to namespace them, so they don't clash with other programs environment variables, which isn't the case here.

Also, how does this interact with buildbot's notion of branch (from a sourcestamp)?