PCA complains when installer is cancelled

RESOLVED FIXED

Status

()

Toolkit
NSIS Installer
P2
normal
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: emk, Assigned: rstrong)

Tracking

(Blocks: 1 bug)

unspecified
x86
Windows 7
Points:
---
Dependency tree / graph
Bug Flags:
blocking1.9.2 -
wanted1.9.2 +
in-testsuite -
in-litmus -

Firefox Tracking Flags

(status1.9.2 .7-fixed, status1.9.1 .11-fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

8 years ago
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
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
I don't recall seeing this when I've installed Firefox or Minefield. What do we need to do to trigger this dialog?
I don't know... the first I heard of this happening was last night via a blog entry.

Updated

8 years ago
Summary: PCA compains when we canceled the installer → PCA complains when installer is cancelled
(Reporter)

Updated

8 years ago
Blocks: 474052

Comment 4

8 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

8 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

8 years ago
Only your first point actually matters. :P
Created attachment 407223 [details]
win32 installer complains to PCA

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)
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: nobody → robert.bugzilla
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+
Duplicate of this bug: 531388
(Reporter)

Updated

7 years ago
Duplicate of this bug: 567497
(Reporter)

Comment 12

7 years ago
This bug will occur even if I didn't cancel the installer (see the duplicate bug for details).
(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.
(Reporter)

Updated

7 years ago
Depends on: 569058
Created attachment 448695 [details] [diff] [review]
patch to update the manifest of the 7zSD.sfx

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)
Attachment #448695 - Attachment description: atch to update the manifest of the 7zSD.sfx → patch to update the manifest of the 7zSD.sfx
Created attachment 448696 [details]
New 7-zip stub
Attachment #448695 - Flags: review?(ted.mielczarek) → review+
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/6a40f493664d

Bug 569058 along with Bug 569534 will fix this for the uninstaller.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Flags: in-testsuite-
Flags: in-litmus-
Resolution: --- → FIXED
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?
cc'ing a few people from SeaMonkey and Thunderbird... you'll want to do this for your 7zSD.sfx files in other-licenses.
Filed bug 569791 for SeaMonkey and attached a patch

Filed Bug 569789 for Thunderbird and attached a patch
Thanks, Rob!

Comment 21

7 years ago
We store the manifest resource in binary in the tree?
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

7 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+
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
You need to log in before you can comment on or make changes to this bug.