1030240 - Add new tree {mozilla,comm}-esr31 to TBPL

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Massimo Gervasini [:massimo]]

Attachment: {mozilla,comm}-esr31 entries.patch

as in bug 808017, needs to be deployed when we have real builds.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8445992 [details] [diff] [review]
{mozilla,comm}-esr31 entries.patch

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

Please make the same changes to mcmerge/js/Config.js as well.

Comment 2

[Massimo Gervasini [:massimo]]

Attachment: {mozilla,comm}-esr31 entries.patch

updated mcmerge/js/Config.js as well.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8446019 [details] [diff] [review]
{mozilla,comm}-esr31 entries.patch

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

We typically list the trees in descending order, so please put the esr31 entries above esr24. Otherwise, looks good with the one other comment addressed.

::: mcmerge/js/Config.js
@@ +274,5 @@
> +    "comm-esr31": {
> +      repo: "releases/comm-esr31",
> +      unconditionalFlag: false,
> +      trackedTree: true,
> +      synonyms: ["releases/comm-esr31", "comm-esr31", "comm-release"]

I have a feeling the comm-release on this line is going to confuse things since it's already on the esr24 line above as well. Let's leave it off for now.

Comment 4

[Nick Thomas [:nthomas] (UTC+12)]

Landed for mgerva:
  https://hg.mozilla.org/webtools/tbpl/rev/4cb36ffa0263

Comment 5

[Ryan VanderMeulen [:RyanVM]]

In production :)