Closed Bug 339714 Opened 14 years ago Closed 14 years ago

Locale repackaging to support the NSIS installer


(Firefox :: Installer, defect, P1)

2.0 Branch
Windows XP



Firefox 2 beta1


(Reporter: robert.strong.bugs, Assigned: benjamin)



(Keywords: fixed1.8.1, Whiteboard: 181b1+)


(5 files, 1 obsolete file)

The build scripts will need to be updated to support locale repackaging for the NSIS installer implemented in bug 326580.
Assignee: nobody → benjamin
Priority: -- → P1
Target Milestone: --- → Firefox 2 beta1
Depends on: 340864
Flags: blocking-firefox2+
Depends on: 340173
This is a prerequisite small patch which separates out the building of setup.exe into a separate target from that which builds the entire installer.
Attachment #227268 - Flags: review?(robert.bugzilla)
Attachment #227268 - Flags: review?(robert.bugzilla) → review+
Attachment #227315 - Flags: review?(robert.bugzilla)
Checked in browser/installer/windows/ change to calendar/installer/windows/ also

Reminder to bsmedberg per IRC discussion to land this on branch eventually, as calendar's NSIS stuff is also on branch at this time.

I wonder what we do about README.txt, which I recall not being in localized, 
and where do the spellcheck dictionaries end up right now?
I guess for now, we would like to keep the en-US dict in there, but that's likely an offline discussion.

Those comments without a windows env, so I haven't applied and tested the patch.
(In reply to comment #4)
> I wonder what we do about README.txt, which I recall not being in localized

Unless I'm looking at the wrong thing, README.txt is localized (although the filename itself is not.)
> Created an attachment (id=227268) [edit]
> Make setup.exe generation separate, rev. 1

With this checkin, I'm now getting errors when building the installer because ABS_CONFIG_DIR is empty, so 7zip is trying to put the app in /app.7zs. 

I'm seeing "/bin/sh: line 0: cd: instgen: No such file or directory" when the $shell command that sets ABS_CONFIG_DIR executes in toolkit/mozapps/installer/windows/nsis/

Whiteboard: 181b1+
patch by bsmedberg, r=lilmatt
Attachment #227347 - Flags: review+
(In reply to comment #7)
> Created an attachment (id=227347) [edit]
> patch to fix regression

Patch checked in on _trunk_only_.
filed bug 342938 on the README.txt issue.
We add it to the localized files at as
part of the libs target.
Comment on attachment 227315 [details] [diff] [review]
Repack it!, rev. 1

very nice
Attachment #227315 - Flags: review?(robert.bugzilla) → review+
Attached patch patch for calendar (obsolete) — Splinter Review
Changes needed for calendar
Attachment #227390 - Flags: review?(mattwillis)
> Created an attachment (id=227390) [edit]
> patch for calendar

-	cp $(topsrcdir)/calendar/EULA.txt instgen/license.txt
+	cp $(topsrcdir)/browser/EULA $(CONFIG_DIR)/license.txt

Um, no.

+APP_VERSION := $(shell cat $(srcdir)/../config/version.txt)

Is this necessary for the patch to work? We've been on a mission to use $SUNBIRD_VERSION rather than catting version.txt everywhere. 

-    -DAB_CD=$(AB_CD) \
-    -DMOZ_LANGPACK_EID=langpack-$(AB_CD) \
-    $(NULL)
+	-DAB_CD=$(AB_CD) \
+	-DMOZ_LANGPACK_EID=langpack-$(AB_CD) \
+	$(NULL)

Pull this bit so as to not hose CVS blame. Besides, you only NEED tabs inside the makefile targets :)
Attachment #227440 - Flags: review?(l10n)
Comment on attachment 227390 [details] [diff] [review]
patch for calendar

Opened bug 343024 for Calendar
Attachment #227390 - Attachment is obsolete: true
Attachment #227390 - Flags: review?(mattwillis)
Comment on attachment 227440 [details] [diff] [review]
Fix branding, rev. 1

looks good to me, and makes sense with the error log we have for ga-IE, too.
Attachment #227440 - Flags: review?(l10n) → review+
To verify that this is fixed, we actually need nsis on the tinderboxens, filed bug
Depends on: 343055
Attachment #227577 - Flags: approval1.8.1?
Comment on attachment 227577 [details] [diff] [review]
1.8 branch merge, rev. 1

approved as per 181drivers
Attachment #227577 - Flags: approval1.8.1? → approval1.8.1+
Whiteboard: 181b1+ → 181b1+ [checkin needed]
Woot, trunk and 1.8
Closed: 14 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: 181b1+ [checkin needed] → 181b1+
You need to log in before you can comment on or make changes to this bug.