1396563 - L10n Dailies broken: Windows since 19-Aug-2017, Mac/Linux since 24-Aug-2017

Status: RESOLVED FIXED

(Thunderbird :: Build Config, defect)

Description

[Jorg K (CEST = GMT+2)]

Windows:

http://ftp.mozilla.org/pub/thunderbird/nightly/2017/09/2017-09-04-03-02-07-comm-central-l10n/comm-central-win32-l10n-nightly-zh-CN-bm74-build1-build2570.txt.gz
says:
IOError: [Errno 2] c:\builds\moz2_slave\tb-c-cen-w32-l10n-ntly-0000000\build\comm-central\mozilla\browser/extensions/shield-recipe-client/test/unit/mock_api/api/v1/action/preference-experiment/implementation/sha384-KQgG38GQ7KZAb2VIB48ANQO6nBcxZoLm2ORzUviRT5nAvSywyPjZ5cJIElw6iXIt: The system cannot find the path specified

Linux/Mac:

http://ftp.mozilla.org/pub/thunderbird/nightly/2017/09/2017-09-04-03-02-07-comm-central-l10n/comm-central-macosx64-l10n-nightly-zh-TW-bm85-build1-build2570.txt.gz
http://ftp.mozilla.org/pub/thunderbird/nightly/2017/09/2017-09-04-03-02-07-comm-central-l10n/comm-central-linux64-l10n-nightly-zh-TW-bm73-build1-build1759.txt.gz

says:
Error: Locale doesn't have a manifest entry for 'devtools-shim'
Traceback (most recent call last):
  File "/builds/slave/tb-c-cen-m64-l10n-ntly-0000000/build/comm-central/mozilla/toolkit/mozapps/installer/l10n-repack.py", line 59, in <module>
    main()
  File "/builds/slave/tb-c-cen-m64-l10n-ntly-0000000/build/comm-central/mozilla/toolkit/mozapps/installer/l10n-repack.py", line 55, in main
    non_resources=args.non_resource, non_chrome=NON_CHROME)
  File "/builds/slave/tb-c-cen-m64-l10n-ntly-0000000/build/comm-central/mozilla/python/mozbuild/mozpack/packager/l10n.py", line 269, in repack
    _repack(app_finder, l10n_finder, copier, formatter, non_chrome)
  File "/tools/python/lib/python2.7/contextlib.py", line 24, in __exit__
    self.gen.next()
  File "/builds/slave/tb-c-cen-m64-l10n-ntly-0000000/build/comm-central/mozilla/python/mozbuild/mozpack/errors.py", line 131, in accumulate
    raise AccumulatedErrors()
mozpack.errors.AccumulatedErrors
make[1]: *** [repackage-zip] Error 1
make: *** [repackage-zip-zh-TW] Error 2

Comment 1

[Jorg K (CEST = GMT+2)]

"L10n Dailies are a constant problem, they break all the time and no one notices" is still a problem, that was the original summary of bug 1385642.

Comment 2

[Axel Hecht [:Pike]]

The devtools shim is bug 1386616, which probably needs a vanilla copy-n-paste patch.

The windows one is a problem with path lengths on windows, I assume.

Comment 3

[Jorg K (CEST = GMT+2)]

Thanks.

I could try to pick bits from https://hg.mozilla.org/mozilla-central/rev/bae931220733 but the browser and mail files don't quite match, for example we don't have these:
https://hg.mozilla.org/mozilla-central/rev/bae931220733#l1.30
https://hg.mozilla.org/mozilla-central/rev/bae931220733#l3.12

The other three hunks could be ported. Of course it would be better if someone did it who knows what they're doing.

Comment 4

[Jorg K (CEST = GMT+2)]

Attachment: 1396563-devtools-shim.patch

Can you please look at this. I didn't port:
https://hg.mozilla.org/mozilla-central/rev/bae931220733#l1.30
https://hg.mozilla.org/mozilla-central/rev/bae931220733#l3.12

Comment 5

[Axel Hecht [:Pike]]

Comment on attachment 8904245 [details] [diff] [review]
1396563-devtools-shim.patch

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

hrmpf, devtools/shim isn't going to work as is, we either need to create an l10n.ini file in devtools/shim/locales (doh) or update compare-locales in m-c to use the toml files. Which will allow to just use devtools/shim as path in mozilla.

I'd like to land compare-locales anyway, but I have to align a few other things first. I'm tracking that in bug 1382005.

The patch goes in the right direction, but the filter.py work is tied to the ini file, and the ini file won't work as it :-/

Comment 6

[Jorg K (CEST = GMT+2)]

Axel, thanks for looking at it. I'm afraid I'll have to leave it to someone one more knowledgable. There's also bug 1392245 in this area.

Comment 8

[Takanori MATSUURA]

(In reply to Axel Hecht [:Pike] from comment #5)
> I'd like to land compare-locales anyway, but I have to align a few other
> things first. I'm tracking that in bug 1382005.

Now that bug 138205 has been fixed.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/c731b22f588d
Port Bug 1382005 to calendar: Update compare-locales invocation; rs=bustage-fix
https://hg.mozilla.org/comm-central/rev/535c47ddb055
Port bug 1386616 to C-C [Move devtools key-shortcuts.properties out of the devtools addon]. rs=bustage-fix

Comment 10

[Jorg K (CEST = GMT+2)]

Hmm, some of it was supposedly my code, I can't remember any of it. I hope you picked the correct hunks ;-) Thanks!

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/5cdc0ea5bca1
Disable configuring stylo for L10N builds; rs=bustage-fix

Comment 12

[Axel Hecht [:Pike]]

Comment on attachment 8904245 [details] [diff] [review]
1396563-devtools-shim.patch

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

::: mail/locales/l10n.toml
@@ +96,5 @@
>      path = "{mozilla}/toolkit/locales/l10n.toml"
>  
> +[[paths]]
> +    reference = "{mozilla}/devtools/shim/locales/en-US/**"
> +    l10n = "{l}{mozilla}/devtools/shim/**"

The l10n path is wrong, follow-up coming.

Comment 13

[Axel Hecht [:Pike]]

Attachment: correct the l10n path for devtools/shim

Comment 14

[Jorg K (CEST = GMT+2)]

Comment on attachment 8916939 [details] [diff] [review]
correct the l10n path for devtools/shim

Thanks for the patch. I'll let Tom deal with this.

Comment 15

[Tom Prince [:tomprince]]

Comment on attachment 8916939 [details] [diff] [review]
correct the l10n path for devtools/shim

Nice catch. Thanks.

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/e6c1d0bc8fa5
Correct the l10n path for devtools/shim; r=tomprince

Comment 17

[Takanori MATSUURA]

linux, linux64, and macosx64 l10n is now available.

However win32 l10n is still unavailable because calendar-wget-en-US is failed.
https://archive.mozilla.org/pub/thunderbird/nightly/2017/10/2017-10-14-03-02-01-comm-central-l10n/comm-central-win32-l10n-nightly-ja-bm74-build1-build2798.txt.gz
https://hg.mozilla.org/comm-central/annotate/706e5fdd301201b23e96be9826b7e5b953d11776/mail/locales/Makefile.in#l200

Comment 18

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Windows is broken on bug 1366607 comment 21, I would think this should be broken for all platforms.

Comment 19

[Axel Hecht [:Pike]]

Yeah, https://ftp.mozilla.org/pub/thunderbird/nightly/2017/10/2017-10-15-04-05-50-comm-central-l10n/comm-central-linux-l10n-nightly-de-bm72-build1-build3142.txt.gz shows that macs are busted, too.

They did create a build on the 9th for the mac.

Comment 20

[Takanori MATSUURA]

(In reply to Philipp Kewisch [:Fallen] from comment #18)
> I would think this should be broken for all platforms.
Sorry. You're right.

Comment 21

[Jorg K (CEST = GMT+2)]

Beta (TB 57):
https://hg.mozilla.org/releases/comm-beta/rev/5e59a63db208ce72b0f482882a20f86149d58b74
https://hg.mozilla.org/releases/comm-beta/rev/338a98536049c8c2b406274780e01f0808496938

Comment 22

[Firefox Bug Husbandry Bot]

Removing leave-open keyword from resolved bugs, per :sylvestre.