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

scoop checkup could check NTFS-ness #1944

Closed
mdaniel opened this issue Jan 6, 2018 · 3 comments
Closed

scoop checkup could check NTFS-ness #1944

mdaniel opened this issue Jan 6, 2018 · 3 comments

Comments

@mdaniel
Copy link

mdaniel commented Jan 6, 2018

  • Current behavior: scoop install $thing fails due to $env:SCOOP living on an ExFAT drive

    scoop checkup did not verify that $env:SCOOP (and presumably its $env:SCOOP_GLOBAL friend) resides on an NTFS volume

  • Expected behavior: scoop checkup would report "Scoop requires an NTFS volume to work" (optionally including to work "to make use of junction links", if the primary audience is technical folks, since I bet that is the clearest)

I have an SSD for my primary volume, and put development stuff (Maven home, most git checkouts, that kind of thing) on the larger, non-Windows volume. For historical, cross-OS reasons, that volume is ExFAT, not NTFS. So, when I pointed $env:SCOOP at d:\Scoop, things did not end well.

PS D:\> scoop install sudo
Scoop was updated successfully!
Installing 'sudo' (0.2017.03.24) [64bit]
sudo.ps1 (2.0 KB) [===========================================================================================] 100%
Checking hash of sudo.ps1... ok.
Linking D:\Scoop\apps\sudo\current => D:\Scoop\apps\sudo\0.2017.03.24
Local NTFS volumes are required to complete the operation.
Creating shim for 'sudo'.
% : Exception calling "GetUnresolvedProviderPathFromPSPath" with "1" argument(s): "Cannot find drive. A drive with the name 'File not found - D' does not exist."
At D:\Scoop\apps\scoop\current\lib\install.ps1:678 char:34
+     $shims | ?{ $_ -ne $null } | % {
+                                  ~~~
+ CategoryInfo          : NotSpecified: (:) [ForEach-Object], MethodInvocationException
+ FullyQualifiedErrorId : DriveNotFoundException,Microsoft.PowerShell.Commands.ForEachObjectCommand
@r15ch13
Copy link
Member

r15ch13 commented Jan 8, 2018

Something like this 43a02b2?

@mdaniel
Copy link
Author

mdaniel commented Jan 8, 2018

You're very kind, thank you for actioning that so quickly!

(As a friendly reminder, you can also close this issue with that commit, unless you were expecting more commits to follow)

@mdaniel mdaniel closed this as completed Jan 8, 2018
@r15ch13
Copy link
Member

r15ch13 commented Jan 8, 2018

@mdaniel Thanks for the friendly reminder. 😉 Just forgot to add it 😄

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

No branches or pull requests

2 participants