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

Use a proper temp file. #1

Merged
merged 2 commits into from
Oct 18, 2015
Merged

Use a proper temp file. #1

merged 2 commits into from
Oct 18, 2015

Conversation

cbcunc
Copy link
Contributor

@cbcunc cbcunc commented Oct 8, 2015

Standalone compiled Matlab programs have a default path wherever they are installed. For most OSes, writing to the program installation directory (\Program Files for Windows; /Applications for OSX) is a privileged operation. For normal users this will fail. Most OSes provide a directory for temporary files generated by user programs, as well as an API to find it. Matlab provides the cross-platform tempdir function for this purpose. A commit in this PR allows plot_google_map to be used in standalone compiled Matlab programs.

cbcunc added 2 commits October 8, 2015 13:32
Standalone compiled Matlab programs have a default path wherever they are installed. For most OSes, writing to the program installation directory (\Program Files for Windows; /Applications for OSX) is a privileged operation. For normal users this will fail. Most OSes provide a directory for temporary files generated by user programs, as well as an API to find it. Matlab provides the cross-platform tempdir function for this purpose. This commit allow plot_google_map to be used in in standalone compiled Matlab programs.
zoharby added a commit that referenced this pull request Oct 18, 2015
Use a proper temp file.
@zoharby zoharby merged commit 07f72be into zoharby:master Oct 18, 2015
@zoharby
Copy link
Owner

zoharby commented Oct 18, 2015

Thanks for the contribution!

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.

2 participants