-
Notifications
You must be signed in to change notification settings - Fork 64
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
Add Path workaround for windows #226
Conversation
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.
Might be worth updating the README, if other users can confirm that this gets the newer versions of MATLAB working on Windows for them
It's likely that the macOS and Linux issues are similar, but they'll need separate fixes since each platform has a different search behavior for libraries.
Another choice is to modify the environment variables using Lines 84 to 86 in f867397
|
That would have to wrap around every function call, because they can dynamically trigger additional |
I wonder if there is a less invasive approach as a bulk modification of the environment variable, as I think has the potential to corrupt other libraries. |
I believe that However, those directories won't be seen by a
|
@topolarity I think we should explore that option in another PR. For now I will merge this since it at least allows us to use MATLAB. @bolognam do you mind also updating the readme? |
See issue #200 for more details, specifically: