Closed Bug 193222 Opened 22 years ago Closed 22 years ago

Update Mac Desktop Get Info string for Mozilla 1.3 final release

Categories

(SeaMonkey :: Build Config, defect)

Other Branch
PowerPC
macOS
defect
Not set
critical

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.3final

People

(Reporter: lpham, Assigned: lpham)

References

()

Details

(Whiteboard: fixed1.3)

Attachments

(1 file)

Need to checkin to MOZILLA_1_3_BRANCH
Target Milestone: --- → mozilla1.3final
Need a=, sr= and r=
Attachment #115397 - Flags: superreview?(leaf)
Attachment #115397 - Flags: review?(jj)
Attachment #115397 - Flags: review?(jj) → review+
Loan, I reviewed your changes to Info.plist. (nsMacVersion.r is no longer part of the build, but the changes are ok) You also need to update the version string in following file: mozilla/xpfe/bootstrap/macbuild/Contents/Resources/English.lproj/InfoPlist.strings This is a file that overwrites some values of Info.plist for localization purposes, but since the affected string ("Mozilla 1.3b, © 1998-2003 The Mozilla Organization") shouldn't be localized, I will remove the corresponding key (CFBundleGetInfoString) from that file on the trunk so we don't have to duplicate this change in the future. Come and see me if you have questions, but please submit a patch for 1.3 final.
JJ, I cannot submit a diff for binary file so I removed 1.3b to 1.3 inside mozilla/xpfe/bootstrap/macbuild/Contents/Resources/English.lproj/InfoPlist.strings CFBundleGetInfoString = "Mozilla 1.3b, © 1998-2003 The Mozilla Organization"; to CFBundleGetInfoString = "Mozilla 1.3, © 1998-2003 The Mozilla Organization"; (Note: I'm using BBEdit to edit this file) Let me know if it looks ok to you. Thanks.
Add twalker to cc list because bugzilla #194082 needs to submit the patch to change mozilla/xpfe/bootstrap/macbuild/Contents/Resources/English.lproj/InfoPlist.strings as well.
Status: NEW → ASSIGNED
same comment as in 194082: r=jj for the change to .../English.lproj/InfoPlist.strings The reason why we cannot diff these .plist files is because they have a unicode 2-byte header which makes cvs think it's a binary file. Bummer, but I don't think there's anything we can do about it.
Attachment #115397 - Flags: superreview?(leaf) → superreview+
Checked all 3 files.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Whiteboard: fixed1.3
verified
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: