Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Uninstaller check doesn't find 32-bit registry keys #197

Closed
ferventcoder opened this issue Mar 23, 2015 · 1 comment
Closed

Uninstaller check doesn't find 32-bit registry keys #197

ferventcoder opened this issue Mar 23, 2015 · 1 comment
Assignees
@ferventcoder
Labels
0 - Backlog Bug Priority_HIGH
Milestone
0.9.9.3

Comments

@ferventcoder
Copy link
Member

Original here - https://groups.google.com/d/msgid/chocolatey/8fad9df8-1f6e-40e4-aa88-cede5663cd4c%40googlegroups.com?utm_medium=email&utm_source=footer

Gist here - https://gist.github.com/aronovgj/5e828facd319da00273a

2015-03-20 16:01:59,756 [DEBUG] -  Preparing uninstall key 'MsiExec.exe /I{{AC76BA86-7AD7-1031-7B44-AB0000000001}}'
2015-03-20 16:01:59,758 [INFO ] -  Skipping auto uninstaller - The application appears to have been uninstalled already by other means.
2015-03-20 16:01:59,759 [DEBUG] -  Searched for install path 'C:\Program Files (x86)\Adobe\Reader 11.0\Reader\' - found? True
2015-03-20 16:01:59,761 [DEBUG] -  Searched for registry key 'HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\{{{{AC76BA86-7AD7-1031-7B44-AB0000000001}}}}' value 'InstallLocation' - found? False
@ferventcoder ferventcoder added this to the 0.9.9.3 milestone Mar 23, 2015
@ferventcoder
Copy link
Member Author

Actually this is likely related to not correctly searching the 32-bit views of the registry

@ferventcoder ferventcoder changed the title Uninstall has issues when braces are involved Uninstaller check doesn't find 32-bit registry keys Mar 29, 2015
ferventcoder added a commit that referenced this issue Mar 29, 2015
@ferventcoder
(GH-197) Uninstaller check for registry existence
6aeaeda 
The check should take into account 32-bit views of the registry that
the key may have been found in when it was created.
ferventcoder added a commit to ferventcoder/choco that referenced this issue Mar 29, 2015
@ferventcoder
Merge branch 'stable'
ce90c21 
* stable:
  (version) 0.9.9.3
  (doc) update nuspec / changelog for release
  (chocolateyGH-211) Use full path to msiexec
  (chocolateyGH-197) Uninstaller check for registry existence
  (chocolateyGH-167) Get-ChocolateyWebFile create directory
  (chocolateyGH-207) Set up variables for powershell scripts
  (chocolateyGH-202)(log) Enhance formatting escaping
  (chocolateyGH-202) Fix formatting on Noop logging
  (chocolateyGH-208) Do not escape curly braces in scripts
  (chocolateyGH-168) Retry .NET Fx if first install fails
  (chocolateyGH-168) .NET Framework 4.0 install egg/chicken
  (chocolateyGH-207) Some pkgs incorrectly use arguments
@ferventcoder ferventcoder modified the milestones: 0.9.9.3, 0.9.9.5 Mar 30, 2015
@ferventcoder ferventcoder self-assigned this Mar 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ferventcoder ferventcoder

Labels
0 - Backlog Bug Priority_HIGH
Projects
None yet
Milestone
0.9.9.3
Development

No branches or pull requests
1 participant
@ferventcoder