Closed
Bug 1225918
Opened 8 years ago
Closed 8 years ago
Firefox installer is referring to Firefox as 'Firefox' (extra quotes)
Categories
(Firefox :: Installer, defect)
Tracking
()
People
(Reporter: gcp, Assigned: froydnj)
References
Details
Attachments
(2 files)
1.30 KB,
patch
|
mshal
:
review+
lizzard
:
approval-mozilla-aurora+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
260.69 KB,
image/png
|
Details |
It looks like some extra quotation marks slipped into the installer scripts. Every screen in the (stub) installer is referring to the product as 'Firefox' instead of just Firefox. It looks pretty awkward and strange.
Reporter | ||
Comment 1•8 years ago
|
||
Reportedly this also affects the program name when shown in the Windows Default Programs dialog, which means it's surely a bug.
Reporter | ||
Comment 2•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/bdfaafcf994b +DEFINES['MOZ_APP_DISPLAYNAME'] = "'%s'" % CONFIG['MOZ_APP_DISPLAYNAME'] Doesn't this add extra quotes?
Updated•8 years ago
|
Flags: needinfo?(nfroyd)
Assignee | ||
Comment 3•8 years ago
|
||
Moving DEFINES from Makefile.in to moz.build in browser/installer/windows/ kept the single quotes for MOZ_APP_DISPLAYNAME. Unfortunately, the patch author missed that those quotes were there for the shell's benefit, not as a part of the defined value. These extraneous quotes look rather unbecoming in the installer. Since mozbuild adds the appropriate quoting to a defined value, we can simply forego the quoting in the moz.build file. I haven't tested this on a Windows machine yet (I'm not even certain I can build and run the installer!), but the reasoning makes some sense...
Attachment #8689144 -
Flags: review?(mshal)
Assignee | ||
Comment 4•8 years ago
|
||
[Tracking Requested - why for this release]: Cosmetic problems in the installer due to bug 1185637.
tracking-firefox42:
--- → ?
tracking-firefox43:
--- → ?
tracking-firefox44:
--- → ?
tracking-firefox45:
--- → ?
Flags: needinfo?(nfroyd)
I have experienced this in every Firefox version at least for 3-4 weeks, but I thought it should be like this.
Updated•8 years ago
|
Attachment #8689144 -
Flags: review?(mshal) → review+
Comment 7•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e665231d446b
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Tracked for 44.
Nathan, should we consider uplifting this fix to Aurora44?
Flags: needinfo?(nfroyd)
Assignee | ||
Comment 10•8 years ago
|
||
(In reply to Ritu Kothari (:ritu) from comment #9) > Nathan, should we consider uplifting this fix to Aurora44? We should fix this in 44 and 43, if we can; 42 already went out the door with this. :( Will write uplift request shortly.
Flags: needinfo?(nfroyd)
Assignee | ||
Comment 11•8 years ago
|
||
Comment on attachment 8689144 [details] [diff] [review] avoid quoting MOZ_APP_DISPLAYNAME when used as a DEFINE Approval Request Comment [Feature/regressing bug #]: Bug 1185637 [User impact if declined]: Unusual quoting in the installer; not significant user impact, but definitely a fit-and-finish sort of bug. [Describe test coverage new/current, TreeHerder]: Not really applicable here. [Risks and why]: Low low risk. [String/UUID change made/needed]: None, this just touches internal build files.
Attachment #8689144 -
Flags: approval-mozilla-beta?
Attachment #8689144 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → nfroyd
Comment 13•8 years ago
|
||
Comment on attachment 8689144 [details] [diff] [review] avoid quoting MOZ_APP_DISPLAYNAME when used as a DEFINE Trivial fix for a recent regression, OK to uplift to aurora and beta.
Attachment #8689144 -
Flags: approval-mozilla-beta?
Attachment #8689144 -
Flags: approval-mozilla-beta+
Attachment #8689144 -
Flags: approval-mozilla-aurora?
Attachment #8689144 -
Flags: approval-mozilla-aurora+
Comment 14•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/a256d40aa813
Comment 15•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/931a13f5c867
Comment 16•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/a256d40aa813
status-b2g-v2.5:
--- → fixed
Comment 17•8 years ago
|
||
I don't think we need to track this, it's so minor, and looks fixed now.
status-b2g-v2.5:
fixed → ---
Updated•8 years ago
|
QA Whiteboard: [good first verify]
Comment 18•8 years ago
|
||
Could not reproduce on firefox 43.0a1 or 45.0a1 build. I am using Windows 10(32 bit OS) X64 based architecture machine. Can you point me to the build to reproduce the bug before verifying?
Comment 19•8 years ago
|
||
(In reply to Chandrakant Dhutadmal from comment #18) > Could not reproduce on firefox 43.0a1 or 45.0a1 build. I am using Windows > 10(32 bit OS) X64 based architecture machine. Can you point me to the build > to reproduce the bug before verifying? Did you use the stub installer? with this build [1] you will see the extra quotes for Nightly right after the installer launches. [1]https://archive.mozilla.org/pub/firefox/nightly/2015/11/2015-11-18-03-02-32-mozilla-central/firefox-45.0a1.en-US.win32.installer-stub.exe
Comment 20•8 years ago
|
||
(In reply to Alexandra Lucinet, QA Mentor [:adalucinet] from comment #19) > (In reply to Chandrakant Dhutadmal from comment #18) > > Could not reproduce on firefox 43.0a1 or 45.0a1 build. I am using Windows > > 10(32 bit OS) X64 based architecture machine. Can you point me to the build > > to reproduce the bug before verifying? > > Did you use the stub installer? with this build [1] you will see the extra > quotes for Nightly right after the installer launches. > > [1]https://archive.mozilla.org/pub/firefox/nightly/2015/11/2015-11-18-03-02- > 32-mozilla-central/firefox-45.0a1.en-US.win32.installer-stub.exe I could find out the issue in 45.0a1. Was confused with the title in the installer Window. It reads 'Nightly' setup. The title mentioned in this bug is 'Firefox' instead. I am attaching the screenshot which I captured (ExtraQuotesIssue-45.0a1Build.png). I hope this is the issue which the reporter of the bug is talking about.
Comment 21•8 years ago
|
||
Comment 22•8 years ago
|
||
(In reply to Chandrakant Dhutadmal from comment #20) > I could find out the issue in 45.0a1. Was confused with the title in the > installer Window. It reads 'Nightly' setup. The title mentioned in this bug > is 'Firefox' instead. I am attaching the screenshot which I captured > (ExtraQuotesIssue-45.0a1Build.png). I hope this is the issue which the > reporter of the bug is talking about. Yes, that's the case here; along with the one from the option to set the browser as default - https://goo.gl/OXmCiQ - which you don't see because you're using Windows 10 and it was intentionally removed on Windows 8+ (bug 1237592 comment 1). What's left for you to do here is to confirm the fix with latest builds. Thanks for your contribution! :)
Comment 23•8 years ago
|
||
This is verified resolved in Firefox 45.0b3 installer (.exe file). Since I do not have permissions to change the staus of the bug, I am commenting here. [testday-20160205]
Reporter | ||
Updated•8 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Updated•8 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•