900585 - Add support for b2g-inbound to mcMerge

Status: RESOLVED FIXED

(Tree Management :: Bugherder, defect)

Description

[Ryan VanderMeulen (PTO, back July 27)]

Right now, mcMerge doesn't know what b2g-inbound is.

Comment 1

[Ryan VanderMeulen (PTO, back July 27)]

Attachment: part 1 - remove repositories that aren't supported in tbpl anymore

Might as well do some cleanup while I'm here. I'm sure eventually we'll tie these separate configs together.

Comment 2

[Ryan VanderMeulen (PTO, back July 27)]

Attachment: part 2 - add b2g-inbound and a couple other supported b2g18 repos

Comment 3

[Ryan VanderMeulen (PTO, back July 27)]

Attachment: part 1 - remove repositories that aren't supported in tbpl anymore

No idea WTF I did there...

Comment 4

[Graeme McCutcheon [:graememcc]]

Comment on attachment 784471 [details] [diff] [review]
part 1 - remove repositories that aren't supported in tbpl anymore

Thanks for the cleanup!

Comment 5

[Graeme McCutcheon [:graememcc]]

Comment on attachment 784470 [details] [diff] [review]
part 2 - add b2g-inbound and a couple other supported b2g18 repos

Review of attachment 784470 [details] [diff] [review]:
-----------------------------------------------------------------

::: mcmerge/js/Config.js
@@ +99,5 @@
> +      repo: "releases/mozilla-b2g18",
> +      unconditionalFlag: true,
> +      trackedTree: true,
> +      synonyms: ["b2g18", "mozilla-b2g18"]
> +    }

Nit: need a trailing comma on the closing brace. Same for the v1.0.1 and 1.1.0hd properties.

@@ -258,5 @@
>        unconditionalFlag: false,
>        trackedTree: true,
>        synonyms: ["releases/comm-esr17", "comm-esr17"]
>      },
> -    "mozilla-b2g18": {

Nit: comm-esr17 is now the final property in the tree info object. Remove its trailing comma.

Comment 6

[Ryan VanderMeulen (PTO, back July 27)]

Pushed with nits addressed. Thanks!
https://hg.mozilla.org/webtools/tbpl/rev/9c722ab4841c
https://hg.mozilla.org/webtools/tbpl/rev/7af46fbdaff0

Comment 7

[Ed Morley [:emorley]]

In production :-)