511889 - uploads for Fennec builds should include langpacks/deb packages

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P3)

Description

[Guillermo López :willyaranda (probably SLOW response)]

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; es-ES; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2
Build Identifier: 

It could be very useful for localizers to have xpis to test the translation of Fennec.

I know that there are localized builds for fennec, both on 1.9.2 and trunk, so I think it is not too difficult to upload those xpis.

Maybe it could be in the same folder, or on a xpi folder inside the folder where builds are put

Reproducible: Always

Comment 1

[Guillermo López :willyaranda (probably SLOW response)]

Trunk localized builds are uploaded to: http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mobile-trunk-l10n/

And for 1.9.2: http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mobile-1.9.2-l10n/

Comment 2

[Armen [:armenzg]]

Axel, is it fine to future it for now? I can get it assigned to me but I will not revisit it until the end of September.

Comment 3

[Axel Hecht [:Pike]]

We do generate those xpis, we apparently just don't upload them. This should be fixed soon, it should really just be a matter of making the upload pieces picking up another item.

Comment 4

[Axel Hecht [:Pike]]

Is there a reason why we're not using upload.py for mobile builds?

Comment 5

[[e-mail] (:bhearsum)]

(In reply to comment #4)
> Is there a reason why we're not using upload.py for mobile builds?

It didn't exist when when they were setup IIRC.

Comment 6

[Axel Hecht [:Pike]]

Morphing this bug to upload not just the tarball, but all of the localized fennec bits.

Once we get debs up, we'll need to upload those, too. Same for whatever turns out to be the wince installer.

Comment 7

[Lukas Blakk [:lsblakk] use ?needinfo]

Futuring until someone has time to pick this up - maybe a q4 goal?

Comment 8

[Axel Hecht [:Pike]]

This is required for shipping Fennec.

Comment 9

[Armen [:armenzg]]

Taking unless another Releng could take it. Any expected delivery time? Next week or can it wait few weeks?

Comment 10

[Aki Sasaki (not active)]

I believe this will just be a change to packageGlob in mobile_l10n_nightly_factory in mobile_master.py.  I can take a look tomorrow.

Comment 11

[Aki Sasaki (not active)]

Attachment: fix bug 502759, which will break xpi uploads

Bug 502759 means that old XPIs and tarballs will sit in the dist/ directory, and upload with the built files.

Assuming we continue with the packageGlob method of uploading, this will be needed.  That depends on the status of https://bugzilla.mozilla.org/show_bug.cgi?id=502759#c1

Not setting the review flag since this seems to be in a state of flux.  But if we need it, I've tested this method in staging.

Patch 1 of 2.

Comment 12

[Aki Sasaki (not active)]

Attachment: add the xpi to maemo/linux fennec desktop l10n repack packageGlob

The second part of the patch.

This is incomplete since I don't know the name or location of the deb, so I haven't added that. (dependent on bug 490482)

However, this will upload the xpi.

Comment 13

[Armen [:armenzg]]

Reassigning as discussed last week on IRC

Comment 14

[Chris Cooper [:coop] (he/him)]

Comment on attachment 400582 [details] [diff] [review]
fix bug 502759, which will break xpi uploads

Just curious why this patch isn't in bug 502759? ;)

Comment 15

[Aki Sasaki (not active)]

Comment on attachment 400582 [details] [diff] [review]
fix bug 502759, which will break xpi uploads

That's what They would be expecting, isn't it? ;-)

Checked in:
http://hg.mozilla.org/build/buildbotcustom/rev/6013eb249c1f

Comment 16

[Aki Sasaki (not active)]

Comment on attachment 400584 [details] [diff] [review]
add the xpi to maemo/linux fennec desktop l10n repack packageGlob

http://hg.mozilla.org/build/buildbot-configs/rev/15d33eb72736

Comment 17

[Aki Sasaki (not active)]

The xpi should upload now. Waiting for the deb.

Comment 18

[Aki Sasaki (not active)]

I'm going to close this out.
l10n deb creation is tracked in bug 490482 ; if/when that gets fixed, the upload should be part of that fix.