Patch mozharness to define b2g_repo and b2g_branch

RESOLVED FIXED

Status

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: sousmangoosta, Assigned: sousmangoosta)

Tracking

unspecified

Firefox Tracking Flags

(firefox43 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

To use taskcluster for community builds, we need to define a non mozilla b2g repo in testing/mozharness/mozharness/mozilla/building/buildb2gbase.py.
Hi Ronald,
Thanks for filling this.

What are you specifically trying to do?
Do you have a sample command? or what needs changing?
Hey Armen,

Sorry for that's not well explained feature.
You can find my try running here : https://treeherder.mozilla.org/#/jobs?repo=try&revision=850ba464e299
And the changes I did there : https://hg.mozilla.org/try/rev/82adda4e9f56
I did this change, because we need to permit someone from community to easily doing builds on taskcluster.
When the device is not know on mozilla git, we need to point out the B2G repo elsewhere.
Posted patch mozharness.diff (obsolete) — Splinter Review
This patch is ok on try server https://treeherder.mozilla.org/#/jobs?repo=try&revision=850ba464e299 and local too
Attachment #8661125 - Flags: review?(armenzg)
Attachment #8661125 - Flags: review?(armenzg) → review+
Change r=jlund to r=armenzg
Attachment #8661125 - Attachment is obsolete: true
Attachment #8661279 - Flags: review+
Keywords: checkin-needed
I will land this.
Keywords: checkin-needed
I've pushed to try first "just in case" (+ the trees are closed).
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c0097f5169bf
NI myself so I won't forget.
Flags: needinfo?(armenzg)
The trees are still closed.
I'm asking checkin-needed for now.
If the trees are open and the patch has not landed by tomorrow I will land this in the morning.
The try push looks good.
Keywords: checkin-needed
Flags: needinfo?(armenzg)
https://hg.mozilla.org/mozilla-central/rev/2f556e07e536
Status: NEW → RESOLVED
Closed: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.