Closed
Bug 904018
Opened 11 years ago
Closed 11 years ago
mcmerge should know about b2g-inbound
Categories
(Tree Management :: Bugherder, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cbook, Assigned: RyanVM)
References
Details
Attachments
(1 file)
|
1.34 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
did a merge from b2g-inbound to mc today and mc merge flags this as Merge from mozilla-inbound to mozilla-central https://tbpl.mozilla.org/mcmerge/?cset=87c1796bc46c which is wrong since this is b2g-inbound. The flags in the bugs works like expected, but to avoid confusion this should be merge from b2g-inbound to mozilla-central
|
Assignee | |
Comment 1•11 years ago
|
||
b2g-inbound support was added to mcMerge in bug 900585. https://mxr.mozilla.org/webtools-central/source/tbpl/mcmerge/js/Config.js#68 I'm wondering if maybe it's hitting the "inbound" synonym for mozilla-inbound first? If so, maybe we should just rearrange b2g-inbound and mozilla-inbound.
|
||
Comment 2•11 years ago
|
||
Yup: https://mxr.mozilla.org/webtools-central/source/tbpl/mcmerge/js/mcMerge.js#138
|
|
Assignee | |
Comment 3•11 years ago
|
||
|
|
||
Updated•11 years ago
|
Attachment #788896 -
Flags: review?(emorley) → review+
|
|
Assignee | |
Comment 4•11 years ago
|
||
https://hg.mozilla.org/webtools/tbpl/rev/385a97ba05df
|
|
||
Comment 5•11 years ago
|
||
In production :-)
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
|
||
Updated•10 years ago
|
Product: Webtools → Tree Management
|
|
||
Updated•10 years ago
|
Component: TBPL → mcMerge
You need to log in
before you can comment on or make changes to this bug.
Description
•