-
Notifications
You must be signed in to change notification settings - Fork 1.4k
BUG: Fix bug with sys_info on Windows #13310
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
Conversation
Tacked on a little fix for #13154 as well, ready for review/merge from my end! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Being on mne main
, I can run mne sys_info
just fine on a non-elevated powershell (i.e., I do not need this PR for it to work). Do I need to call something special to reproduce the error?
Not 100% sure what needs to be done but without this change this is what I see:
|
These seem to have relevant info:
So on my machine I have:
I can confirm that setting:
allows things to work without this change, and
brings back the error above on |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That makes sense, I have the ExecutionPolicy that allows things to work without this change. This is on Windows 11.
Co-authored-by: Stefan Appelhoff <stefan.appelhoff@mailbox.org>
Fixed a little bug I hit on Windows where not having permissions to call a command-line function caused a ValueError
Closes #13154