Closed Bug 1373374 Opened 7 years ago Closed 7 years ago

SeaMonkey is incorrectly enclosed in quotation marks in Set Default Programs dialog

Categories

(SeaMonkey :: Installer, defect)

All
Windows
defect
Not set
normal

Tracking

(seamonkey2.48 wontfix, seamonkey2.49esr fixed, seamonkey2.50 wontfix, seamonkey2.51 wontfix, seamonkey2.52 wontfix, seamonkey2.54 fixed, seamonkey2.55 fixed, seamonkey2.53 fixed)

RESOLVED FIXED
seamonkey2.55
Tracking Status
seamonkey2.48 --- wontfix
seamonkey2.49esr --- fixed
seamonkey2.50 --- wontfix
seamonkey2.51 --- wontfix
seamonkey2.52 --- wontfix
seamonkey2.54 --- fixed
seamonkey2.55 --- fixed
seamonkey2.53 --- fixed

People

(Reporter: frg, Assigned: frg)

Details

(Whiteboard: SM2.53.1)

Attachments

(2 files, 2 obsolete files)

Tile says it all.

SeaMonkey (Mail) and SeaMonkey (News) are not affected.
Attached image Capture.PNG
picture showing the problem.
Attached patch 1373374-quotationmarks.patch (obsolete) — Splinter Review
SeaMonkey currently is the only product defining them in moz.build so align the installers with TB and IB.
Attachment #8878185 - Flags: review?(rsx11m.pub)
REPRODUCIBLE with Installation of unofficial (by FRG) DE SeaMonkey 2.49.1  (NT 6.1; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0 Build 20170511042150  (Default Classic Theme) on German WIN7 64bit, I see those single quotation marks around SeaMonkey. But I can't tell which installation caused this mistake.
> But I can't tell which installation caused this mistake.

Must have been so for ages in moz.build. See "'%s'" in 

> DEFINES['MOZ_APP_DISPLAYNAME'] = "'%s'" % CONFIG['MOZ_APP_DISPLAYNAME']
Attachment #8878185 - Flags: review?(rsx11m.pub) → review?(iann_bugzilla)
Comment on attachment 8878185 [details] [diff] [review]
1373374-quotationmarks.patch

The correct thing to do would be to port Bug 1225918 - avoid quoting MOZ_APP_DISPLAYNAME when used as a DEFINE
Attachment #8878185 - Flags: review?(iann_bugzilla) → review-
Attached patch 1373374-quotationmarks-V2.patch (obsolete) — Splinter Review
Made the patch simpler :)
Attachment #8878185 - Attachment is obsolete: true
Comment adjusted. Tested and working.

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: cosmetic
Testing completed (on m-c, etc.):c-b 
Risk to taking this patch (and alternatives if risky): low/no risk
String changes made by this patch: none
Attachment #8904074 - Attachment is obsolete: true
Attachment #8904143 - Flags: review?(iann_bugzilla)
Attachment #8904143 - Flags: approval-comm-release?
Attachment #8904143 - Flags: approval-comm-esr52?
Attachment #8904143 - Flags: approval-comm-beta?
Comment on attachment 8904143 [details] [diff] [review]
1373374-quotationmarks-V2.patch

r/a=me
Attachment #8904143 - Flags: review?(iann_bugzilla)
Attachment #8904143 - Flags: review+
Attachment #8904143 - Flags: approval-comm-release?
Attachment #8904143 - Flags: approval-comm-release+
Attachment #8904143 - Flags: approval-comm-esr52?
Attachment #8904143 - Flags: approval-comm-esr52+
Attachment #8904143 - Flags: approval-comm-beta?
Attachment #8904143 - Flags: approval-comm-beta+
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/4415a4c2c949
Remove single quotation marks around MOZ_APP_DISPLAYNAME in SeaMonkey installer. r=IanN
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: SM2.53.1
You need to log in before you can comment on or make changes to this bug.