Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Uninstaller doesn't remove Brackets start menu shortcut (sometimes) #1473

Closed
tvoliter opened this issue Aug 28, 2012 · 18 comments
Closed

Uninstaller doesn't remove Brackets start menu shortcut (sometimes) #1473

tvoliter opened this issue Aug 28, 2012 · 18 comments

Comments

@tvoliter
Copy link
Contributor

  1. Double click "Brackets Sprint 13.msi" file to install. Complete installation
  2. Launch Brackets Sprint 13 from start menu to verify it runs
  3. Double click "Brackets Sprint 13.msi" again. Choose "remove program"

Result:
After uninstall "Brackets Sprint 13" still appears in Programs list. Attempting to launch installed Brackets shows that it's an alias pointing to a missing file

@ghost ghost assigned peterflynn Aug 28, 2012
@tvoliter
Copy link
Contributor Author

I tested this same scenario removing Brackets via Add/Remove Programs and that method also left the "Brackets Sprint 13" alias in the Start menu

@gruehle
Copy link
Member

gruehle commented Aug 28, 2012

Reviewed.

@peterflynn
Copy link
Member

I can't repro via either method -- I tried on the installer build machine (Win 7?) and my own machine (Win XP).

However, Justin says he can repro (in Brackets and Shadow) and thinks he has a fix.

@peterflynn
Copy link
Member

Raymond can repro too. Justin's fix didn't pan out, so we will have to try a different approach...

@peterflynn
Copy link
Member

More updates: Raymond and Justin both can't repro it anymore. Jay can repro on his Win7 machine, though.

@peterflynn
Copy link
Member

I suspect this has something to do with the use of "advertised" vs. "non-advertised" shortcuts, and their interaction with multi-user profile configurations.

We're currently using a "non-advertised" shortcut, which is less fancy but apparently has lots of gotchas around multi-user installs. "Advertised" shortcuts seem more complex and have a few limitations, but are supposedly safer in most cases. Yet I'm pretty sure Raj specifically avoided using an "Advertised" shortcut. I don't know why.

All of that makes me hesitant to crack open this can of worms before Raj returns...

Further reading:

@peterflynn
Copy link
Member

Also UTR for Adam on Windows 8...

@peterflynn
Copy link
Member

Moving out of sprint 13 per discussion. I'll add this to the release notes.

@peterflynn
Copy link
Member

Additional notes:

I thought this might be related to whether the user is running as an admin or not, but it doesn't seem to be. Jay is running as admin and he repros every time; but Adam and the build machine are also running as admin and are never able to repro.

Raymond mentioned that the one time he was able to repro this, he was running as admin and the shortcut was in his user-specific Start menu folder (not the global cross-user one). I'm not sure if that's as expected for non-advertised shortcuts or not, though.

@peterflynn
Copy link
Member

I repro'ed this once on my XP machine, finally. It was while doing a test of side-by-side installs: install sprint 13, launch it, close it, install sprint "14" fake installer, launch it, close it, uninstall sprint 13 -- leaves the shortcut behind. (Uninstalling sprint "14" after that does remove the sprint "14" shortcut... but the 13 one still remains).

@peterflynn
Copy link
Member

Moving to Sprint 15 after discussion

@RajNarayana
Copy link

I have tried few times installing and uninstalling brackets from brackets-sprint-14-WINDOWS build and I couldn't repro this bug.

@peterflynn
Copy link
Member

@RajNarayana: It definitely reproduces with Sprint 14 on several computers (including the installer build machine last I checked). It might be helpful to ping some others on the team in SF to see if they can still reproduce it also.

@peterflynn
Copy link
Member

Changing tag to Sprint 16: we would still like to investigate right away (i.e. start of next week), but we may not have time to merge a fix before the sprint ends.

@peterflynn
Copy link
Member

Moving installer bugs to Sprint 17

@peterflynn
Copy link
Member

Raj suggested we could fix this by making sure that every installer build uses a unique GUID for the StartMenuShortcut component. Despite being set for side-by-side installs, Wix may be trying to ref-count this, which would cause this bug at least for cases where two installations have coexisted.

I suggest we try a manual fix for the upcoming sprint, and if it seems to fix it for everyone then consider automating the process (and close this bug, of course).

@peterflynn
Copy link
Member

Marking FBNC since, as hoped, we'd received no reports of this occurring with the Sprint 16 build.

I've added a Beginning of Sprint task to update the shortcut GUID, and I'll update the installer README's list of "bumping the version number" instructions.

@peterflynn
Copy link
Member

Still appears fixed in the Sprint 17 RC build too, so I think that's enough evidence to close this out now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants