Closed
Bug 522065
Opened 15 years ago
Closed 15 years ago
PCA complains when installer is cancelled
Categories
(Firefox :: Installer, defect, P2)
Tracking
()
RESOLVED
FIXED
People
(Reporter: emk, Assigned: robert.strong.bugs)
References
Details
Attachments
(3 files)
51.07 KB,
image/png
|
Details | |
80.11 KB,
patch
|
ted
:
review+
christian
:
approval1.9.2.7+
christian
:
approval1.9.1.11+
|
Details | Diff | Splinter Review |
119.50 KB,
application/octet-stream
|
Details |
Steps to reproduce:
1. Run installer on Windows 7.
2. Cancel the installer.
Actual result:
Program Compatibility Assistant will pop up.
Expected result:
PCA should not pop up.
We need to update manifest to Windows 7 format to satisfy PCA.
http://forums.winamp.com/showthread.php?postid=2525010
Assignee | ||
Comment 1•15 years ago
|
||
Jim, Vlad, and Rob... just a heads up that firefox.exe might need this added as well.
http://code.msdn.microsoft.com/Windows7AppQuality
download the current release available towards the top on the right
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
<compatibility xmlns="urn:schemas-microsoft-com:compatibility.v1">
<application>
<!—- The ID below indicates application support for Windows Vista -->
<supportedOS Id="{e2011457-1546-43c5-a5fe-008deee3d3f0}"/>
<!—- The ID below indicates application support for Windows 7 -->
<supportedOS Id="{35138b9a-5d96-4fbd-8e2d-a2440225f93a}"/>
</application>
</compatibility>
</assembly>
Flags: blocking1.9.2?
Priority: -- → P2
Comment 2•15 years ago
|
||
I don't recall seeing this when I've installed Firefox or Minefield. What do we need to do to trigger this dialog?
Assignee | ||
Comment 3•15 years ago
|
||
I don't know... the first I heard of this happening was last night via a blog entry.
Updated•15 years ago
|
Summary: PCA compains when we canceled the installer → PCA complains when installer is cancelled
Reporter | ||
Updated•15 years ago
|
Blocks: win7support
Comment 4•15 years ago
|
||
I believe it only happens if the executable looks like an installer ("setup" in the name, etc.) and exits *without* adding a Programs and Features entry (may also count updating). A successful install won't trigger it. AFAIK it just suggests re-launching elevated, guessing that the install most likely failed because it didn't elevate.
As long as Firefox's setup elevates by itself (either at the start due to manifest or during the process) when installing for all users, it can be marked compatible with Windows 7 to avoid the PCA. However, if no elevation occurs w/o the user explicitly requesting it (e.g. via Explorer's context menu) then I'd strongly suggest against marking it compatible, since the PCA will be helping people get Firefox installed in that case.
I don't believe Firefox itself would benefit from being marked compatible given the list of things that affects, but that can be infestigated seperately.
Reporter | ||
Comment 5•15 years ago
|
||
Firefox installer asks elevation by itself. We should not rely on the compatibility feature which can be turned off.
Moreover, users can cancel the setup at their own will (as I showed in STR). PCA dialog will confuse them in that case.
Comment 6•15 years ago
|
||
Only your first point actually matters. :P
Comment 7•15 years ago
|
||
Rob, here the screen of the PCA on W7 trying to install hourly win32.installer.exe files.
I get the PCA error even after setup.exe launches Minefield (firefox.exe) just fine. I close Minefield, then the PCA pops up just for the installer.exe (no mention of the setup.exe or firefox.exe)
Comment 8•15 years ago
|
||
Blocking for investigation; rs, I'll leave it up to you to decide if this is a common issue or not. I don't see what the reporter says is happening, but I also don't cancel installs ;)
Flags: blocking1.9.2? → blocking1.9.2+
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → robert.bugzilla
Comment 9•15 years ago
|
||
Cancelling installs for many installers pops up that dialog, for the reasons that Silver outlines. It's confusing, but it's not really our UI, and I don't think we would hold the release for it. Even "wanted" seems like a stretch given our schedule and load at this point, but I'm in a stretchy mood.
Flags: wanted1.9.2+
Flags: blocking1.9.2-
Flags: blocking1.9.2+
Reporter | ||
Comment 12•15 years ago
|
||
This bug will occur even if I didn't cancel the installer (see the duplicate bug for details).
Comment 13•15 years ago
|
||
(In reply to comment #12)
> This bug will occur even if I didn't cancel the installer (see the duplicate
> bug for details).
Confirming, as I was seeing a similar experience. Though I haven't seen it much since my first comment 7. Using Win7 32bit.
Assignee | ||
Comment 14•15 years ago
|
||
This will take care of the installer portion... we'll still want to fix bug 569058.
This changes the manifest resource to
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
<assemblyIdentity version="1.0.0.0" processorArchitecture="X86" name="7zS.sfx.exe" type="win32"/>
<description>7-Zip Self-extracting Archive v4.42</description>
<dependency>
<dependentAssembly>
<assemblyIdentity type="win32" name="Microsoft.Windows.Common-Controls" version="6.0.0.0" processorArchitecture="X86" publicKeyToken="6595b64144ccf1df" language="*"/>
</dependentAssembly>
</dependency>
<trustInfo xmlns="urn:schemas-microsoft-com:asm.v3">
<security>
<requestedPrivileges><requestedExecutionLevel level="asInvoker" uiAccess="false"/>
</requestedPrivileges>
</security>
</trustInfo>
<compatibility xmlns="urn:schemas-microsoft-com:compatibility.v1">
<application>
<supportedOS Id="{35138b9a-5d96-4fbd-8e2d-a2440225f93a}"/>
<supportedOS Id="{e2011457-1546-43c5-a5fe-008deee3d3f0}"/>
</application>
</compatibility>
</assembly>
Attachment #448695 -
Flags: review?(ted.mielczarek)
Assignee | ||
Updated•15 years ago
|
Attachment #448695 -
Attachment description: atch to update the manifest of the 7zSD.sfx → patch to update the manifest of the 7zSD.sfx
Assignee | ||
Comment 15•15 years ago
|
||
Updated•15 years ago
|
Attachment #448695 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 16•15 years ago
|
||
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/6a40f493664d
Bug 569058 along with Bug 569534 will fix this for the uninstaller.
Assignee | ||
Updated•15 years ago
|
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite-
Flags: in-litmus-
Resolution: --- → FIXED
Assignee | ||
Comment 17•15 years ago
|
||
Comment on attachment 448695 [details] [diff] [review]
patch to update the manifest of the 7zSD.sfx
Drivers, this updates the 7-zip stub manifest for Windows 7 and we should take it on the branches
Attachment #448695 -
Flags: approval1.9.2.5?
Attachment #448695 -
Flags: approval1.9.1.11?
Assignee | ||
Comment 18•15 years ago
|
||
cc'ing a few people from SeaMonkey and Thunderbird... you'll want to do this for your 7zSD.sfx files in other-licenses.
Assignee | ||
Comment 19•15 years ago
|
||
Filed bug 569791 for SeaMonkey and attached a patch
Filed Bug 569789 for Thunderbird and attached a patch
Comment 20•15 years ago
|
||
Thanks, Rob!
Comment 21•14 years ago
|
||
We store the manifest resource in binary in the tree?
Assignee | ||
Comment 22•14 years ago
|
||
It is a resource in the 7-zip self extracting archive itself and we update it when necessary using reshack.exe since we don't compile it at build time. Examples
http://hg.mozilla.org/mozilla-central/log/7e685b61a291/other-licenses/7zstub/firefox/7zSD.sfx
http://bonsai.mozilla.org/cvslog.cgi?file=mozilla/other-licenses/7zstub/firefox/7zSD.sfx&rev=HEAD&mark=1.5
Comment 23•14 years ago
|
||
Comment on attachment 448695 [details] [diff] [review]
patch to update the manifest of the 7zSD.sfx
a=LegNeato for 1.9.2.6 and 1.9.1.11. Please land this on mozilla-1.9.2 default and mozilla-1.9.1 default.
Code freeze is this Friday @ 11:59 pm PST.
Attachment #448695 -
Flags: approval1.9.2.6+
Attachment #448695 -
Flags: approval1.9.2.5?
Attachment #448695 -
Flags: approval1.9.1.11?
Attachment #448695 -
Flags: approval1.9.1.11+
Assignee | ||
Comment 24•14 years ago
|
||
Pushed to mozilla-1.9.1
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/c233c2334bf5
Pushed to mozilla-1.9.2
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/23386e833e53
For posterity I accidentally landed on GECKO1924_20100413_RELBRANCH and backed it out
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/90340fd7a2e6
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/a7c7a3ac2112
status1.9.1:
--- → .11-fixed
status1.9.2:
--- → .6-fixed
Updated•1 year ago
|
Component: NSIS Installer → Installer
Product: Toolkit → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•