376878 - Update verification workaround for duplicate all-l10n.js in Thunderbird2 windows installer

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

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

Attachment: Fix unpack order

See 376877 for the details, but the short version is that two all-l10n.js are included in the windows installer for Thunderbird. On applying the complete update the diff between old and new "install" directories reports:

< //@line 36 "/cygdrive/c/builds/tinderbox/Tb-Mozilla1.8-l10n-Release/WINNT_5.2_Depend/mozilla/../l10n/ca/mail/all-l10n.js"
< 
< //@line 38 "/cygdrive/c/builds/tinderbox/Tb-Mozilla1.8-l10n-Release/WINNT_5.2_Depend/mozilla/../l10n/ca/mail/all-l10n.js"
< 
< pref("general.useragent.locale", "ca");
---
> //@line 36 "/cygdrive/e/builds/tinderbox/Tb-Mozilla1.8-Release/WINNT_5.0_Depend/mozilla/mail/locales/en-US/all-l10n.js"
> 
> //@line 38 "/cygdrive/e/builds/tinderbox/Tb-Mozilla1.8-Release/WINNT_5.0_Depend/mozilla/mail/locales/en-US/all-l10n.js"
> 
> pref("general.useragent.locale", "en-US");
> pref("spellchecker.dictionary", "en-US");

Which is erroneous because the same is not true when you try to reproduce this using the app itself (forcing a complete update).

It turns out the update verification script (unpack.sh) does a 

  7z x <installer>
  cp -rp localized/* nonlocalized/* optional/* bin/

to simulate an install. This patch explicitly sets the order of copy for the localized and nonlocalized dirs, and avoids any confusion by doing the operations sequentially.

Comment 1

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

Checked in.