Closed
Bug 1204630
Opened 10 years ago
Closed 10 years ago
Patch mozharness to define b2g_repo and b2g_branch
Categories
(Release Engineering :: Applications: MozharnessCore, defect)
Release Engineering
Applications: MozharnessCore
Tracking
(firefox43 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: sousmangoosta, Assigned: sousmangoosta)
Details
Attachments
(1 file, 1 obsolete file)
1.68 KB,
patch
|
sousmangoosta
:
review+
|
Details | Diff | Splinter Review |
To use taskcluster for community builds, we need to define a non mozilla b2g repo in testing/mozharness/mozharness/mozilla/building/buildb2gbase.py.
Comment 1•10 years ago
|
||
Hi Ronald,
Thanks for filling this.
What are you specifically trying to do?
Do you have a sample command? or what needs changing?
Assignee | ||
Comment 2•10 years ago
|
||
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
Assignee | ||
Comment 3•10 years ago
|
||
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.
Assignee | ||
Comment 4•10 years ago
|
||
This patch is ok on try server https://treeherder.mozilla.org/#/jobs?repo=try&revision=850ba464e299 and local too
Attachment #8661125 -
Flags: review?(armenzg)
Updated•10 years ago
|
Attachment #8661125 -
Flags: review?(armenzg) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Change r=jlund to r=armenzg
Attachment #8661125 -
Attachment is obsolete: true
Attachment #8661279 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 7•10 years ago
|
||
I've pushed to try first "just in case" (+ the trees are closed).
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c0097f5169bf
Comment 9•10 years ago
|
||
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
Updated•10 years ago
|
Flags: needinfo?(armenzg)
Comment 10•10 years ago
|
||
Comment 11•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox43:
--- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•