566932 - maemo5-gtk l10n repacks break on tar

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P4)

Description

[Aki Sasaki [:aki] (he/him) (back Mon Oct 10) (UTC-6)]

/home/cltbld/build/gtk-maemo5-mobile-1.9.2-l10n/mozilla-1.9.2/config/nsinstall -D fy-NL/tmp
cd ../../dist/xpi-stage/locale-fy-NL && \
	  tar --exclude=install.rdf --exclude=chrome.manifest --exclude=crashreporter.app -cvhf - * | ( cd /home/cltbld/build/gtk-maemo5-mobile-1.9.2-l10n/mozilla-1.9.2/mobile/locales/data-stage//usr/local/lib/fennec-1.1b2pre && tar -xf - )
/scratchbox/tools/bin/sh: line 2: cd: /home/cltbld/build/gtk-maemo5-mobile-1.9.2-l10n/mozilla-1.9.2/mobile/locales/data-stage//usr/local/lib/fennec-1.1b2pre: No such file or directory
chrome/
chrome/fy-NL.jar
tar: -: Cannot write: Broken pipe
tar: Error is not recoverable: exiting now
make[1]: *** [repackage-deb] Error 1
make[1]: Leaving directory `/home/cltbld/build/gtk-maemo5-mobile-1.9.2-l10n/mozilla-1.9.2/mobile/locales'
make: *** [deb-fy-NL] Error 2

I'm going to guess that /usr/local/lib/fennec-1.1b2pre is no longer accurate for maemo5-gtk =P

Comment 1

[Aki Sasaki [:aki] (he/him) (back Mon Oct 10) (UTC-6)]

Attachment: use pf.get('enUS_binaryURL',...

This is already checked into staging. I'd verify again in staging but all envs are taken.

This should get maemo5-gtk l10n repacks to at least download the bits from the right directory.

Comment 2

[Aki Sasaki [:aki] (he/him) (back Mon Oct 10) (UTC-6)]

Comment on attachment 447205 [details] [diff] [review]
use pf.get('enUS_binaryURL',...

Just noticed that this isn't hacking l10nUploadPath.

Yay loop + not spelling this stuff out.

Comment 3

[Aki Sasaki [:aki] (he/him) (back Mon Oct 10) (UTC-6)]

Attachment: new shinier patch

Ok. This patch:

- removes the unused l10nUploadPath
- hacks the baseUploadDir for maemo5 l10n
- hacks the enUSBinaryURL for maemo5 l10n (already done in staging)

so it should fix where the deb is downloading from (latest-mobile-trunk-maemo5-gtk rather than latest-mobile-trunk) and where it uploads l10n repacks to (latest-mobile-trunk-maemo5-gtk-l10n rather than latest-mobile-trunk-l10n).

All the staging envs are taken or sans-slaves, so untested past test-masters.sh/checkconfig. However, all maemo5-gtk repacks are currently busted on production, so up to you if you want to block on that.

Comment 4

[Aki Sasaki [:aki] (he/him) (back Mon Oct 10) (UTC-6)]

Comment on attachment 447224 [details] [diff] [review]
new shinier patch

http://hg.mozilla.org/build/buildbot-configs/rev/7a2d24b8dc58

Comment 5

[Aki Sasaki [:aki] (he/him) (back Mon Oct 10) (UTC-6)]

Yay!
http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mobile-trunk-maemo5-gtk-l10n/

... I also had to clobber all the linux l10n slaves on pm02 for this to work.