Closed Bug 299390 Opened 19 years ago Closed 19 years ago

Hack around unexpected files in tbird localization build process

Categories

(Toolkit Graveyard :: Build Config, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.8beta3

People

(Reporter: benjamin, Assigned: benjamin)

Details

Attachments

(1 file, 1 obsolete file)

rm -f -rf ../../dist/l10n-stage
mkdir ../../dist/l10n-stage
cd ../../dist/l10n-stage &&
/cygdrive/c/builds/tinderbox/Tb-Trunk-l10n/WINNT_5.1_Clobber/mozilla/build/cygwin-wrapper
7z e
/cygdrive/c/builds/tinderbox/Tb-Trunk-l10n/WINNT_5.1_Clobber/mozilla/dist/install/sea/thunderbird-1.0+.en-US.win32.installer.exe

7-Zip 3.12  Copyright (c) 1999-2003 Igor Pavlov  2003-12-10

Processing archive:
c:/builds/tinderbox/Tb-Trunk-l10n/WINNT_5.1_Clobber/mozilla/dist/install/sea/thunderbird-1.0+.en-US.win32.installer.exe

file setuprsc.dll
already exists. Overwrite with 
setuprsc.dll?
(Y)es / (N)o / (A)lways / (S)kip all / A(u)to rename / (Q)uit? file setuprsc.dll
already exists. Overwrite with 
setuprsc.dll?
(Y)es / (N)o / (A)lways / (S)kip all / A(u)to rename / (Q)uit? file setuprsc.dll
already exists. Overwrite with 
setuprsc.dll?
(Y)es / (N)o / (A)lways / (S)kip all / A(u)to rename / (Q)uit? 

System error:
Unspecified error

make[2]: *** [repackage-win32-installer] Error 2

This is really bizarre, because the nice rm -rf above should have removed
anything from this dir. Is something holding onto setuprsc.dll so that "rm"
can't remove it? Anyway, I would like as a hack to add the "-y" switch to 7zip
to keep going after this error.
Attached patch use 7z e -y (obsolete) — Splinter Review
Attachment #187958 - Flags: first-review?(chase)
Need to resolve this so that we have tbird windows builds well before b4.
Flags: blocking1.8b4+
Comment on attachment 187958 [details] [diff] [review]
use 7z e -y

This is per RL conversation between myself and bsmedberg.  Need to find out why
this file is present to begin with prior to attempting to extract.
Attachment #187958 - Flags: first-review?(chase) → first-review+
The problem was a lack of backslash command continuers. Stupid, stupid,
stupid!!!
Attachment #187958 - Attachment is obsolete: true
Checked in.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.8beta3
Product: Toolkit → Toolkit Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: