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.

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

Fix incomplete cleanup/uninstall of VMware Horizon Client #43

Merged
merged 1 commit into from
Jul 8, 2019
Merged

Fix incomplete cleanup/uninstall of VMware Horizon Client #43

merged 1 commit into from
Jul 8, 2019

Conversation

beargle
Copy link
Contributor

@beargle beargle commented Jul 8, 2019

Error

PS> scoop cleanup vmware-horizon-client-np
Removing vmware-horizon-client-np: 5.0.0-12606690
Remove-Item : Access to the path 'C:\Users\name\scoop\apps\vmware-horizon-client-np\5.0.0-12606690\files' is denied.
At C:\Users\name\scoop\apps\scoop\current\libexec\scoop-cleanup.ps1:51 char:9
+         Remove-Item $dir -ErrorAction Stop -Recurse -Force
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : PermissionDenied: (C:\Users\name\sc\u2026.0.0-12606690\files:DirectoryInfo) [Remove-Item], UnauthorizedAccessException
+ FullyQualifiedErrorId : RemoveFileSystemItemUnAuthorizedAccess,Microsoft.PowerShell.Commands.RemoveItemCommand

Fix

During install, a subdirectory of scoop application version $dir is defined to workaround an issue with manifest.json creation in non-elevated shell. Uninstall script should remove this subdirectory to ensure scoop cleanup/uninstall can complete successfully.

During install, a subdirectory of scoop application version $dir is
defined to workaround an issue with 'manifest.json' creation in
non-elevated shell. Uninstall script should remove this subdirectory to
ensure scoop cleanup/uninstall can complete successfully.
@oltolm oltolm merged commit 781fc44 into TheRandomLabs:master Jul 8, 2019
@beargle beargle deleted the fix/vmware-horizon-client-np-uninstall branch July 9, 2019 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants