1193620 - desktop l10n releases require 'export' in 'make config'

Status: RESOLVED FIXED

(Release Engineering :: Release Automation: Other, defect)

Description

[Jordan Lund (:jlund)]

similar to https://bugzil.la/1169937

we are hitting this now because this export requirement was in aurora and has uplifted to beta

Comment 1

[Jordan Lund (:jlund)]

Attachment: 150811_1193620_add_export_to_release_beta_l10n_make_config.patch

this will have to be patched if successful to remove beta specific stuff. I have the beta check in so I, hopefully, don't mess with current 40.0.1 release build in flight since this code is shared.

also note, this should fix TB too (or at least affect TB)

Comment 2

[Jordan Lund (:jlund)]

Comment on attachment 8646761 [details] [diff] [review]
150811_1193620_add_export_to_release_beta_l10n_make_config.patch

thanks: https://hg.mozilla.org/build/tools/rev/44d23de5f9a0

also moved tags of 41 to accept this new rev

Comment 3

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

You had a plan for how to handle this when 41.0 release rolls around ?

Comment 4

[Jordan Lund (:jlund)]

Comment on attachment 8646761 [details] [diff] [review]
150811_1193620_add_export_to_release_beta_l10n_make_config.patch

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

::: lib/python/build/l10n.py
@@ +91,5 @@
>                      cwd=path.join(sourceRepoName, objdir, path.dirname(dir)),
>                      env=env)
>          else:
> +            target = []
> +            if path.basename(dir) == 'config' and sourceRepoName.endswith('mozilla-beta'):

I *think* we can simply remove ` and sourceRepoName.endswith('mozilla-beta')`

the question is whether we can do that now which would effect any 40 related chemspills.

Comment 5

[Jordan Lund (:jlund)]

Attachment: 150827_1193620_l10n_desk_requires_export_in_make-comm-beta-fix_tools.patch

shrinking suffix to allow for comm-beta

Comment 6

[Jordan Lund (:jlund)]

Comment on attachment 8653772 [details] [diff] [review]
150827_1193620_l10n_desk_requires_export_in_make-comm-beta-fix_tools.patch

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

thanks, checked in and updated desktop ff beta5 tags

Comment 7

[Jordan Lund (:jlund)]

(In reply to Jordan Lund (:jlund) from comment #6)
> Comment on attachment 8653772 [details] [diff] [review]
> 150827_1193620_l10n_desk_requires_export_in_make-comm-beta-fix_tools.patch
> 
> Review of attachment 8653772 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> thanks, checked in and updated desktop ff beta5 tags

this time, I updated the tb beta1 tags (the ones I should have done):
https://hg.mozilla.org/build/tools/rev/ee7f685c5ebb

Comment 8

[Jordan Lund (:jlund)]

Attachment: 150913_1193620_remove_beta_logic_from_make_export-tools.patch

time to land. uplift is tomorrow.

Comment 9

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

Comment on attachment 8660499 [details] [diff] [review]
150913_1193620_remove_beta_logic_from_make_export-tools.patch

I'll just take this for a spin on mozilla-esr38 ...

Comment 10

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

Comment on attachment 8660499 [details] [diff] [review]
150913_1193620_remove_beta_logic_from_make_export-tools.patch

A Thunderbird l10n build based on comm-esr38/mozilla-esr38 was green this patch. When you land, please update the comment to remove the reference to 'beta'.