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

create new_sysimage_source_path in temp folder #367

Merged
merged 1 commit into from Mar 7, 2020

Conversation

matsievskiysv
Copy link
Contributor

When using Julia from Debian repository, base_dir is a write protected folder /usr/share/julia/base/. So, when creating a new app with create_app(), it fails due to lack of privileges. This PR moves temporary new_sysimage_source_path to the tmp directory, as it probably was intended.

@KristofferC
Copy link
Sponsor Member

This is not correct, the current setup requires it to be in the same folder as the rest of the Base files.

@matsievskiysv
Copy link
Contributor Author

Well, it worked on my machine. The only place new_sysimage_source_path is used is in this call, where it's passed explicitly. After that it is deleted.

@matsievskiysv
Copy link
Contributor Author

Just tested with sysimage.jl generated by create_app. It's not necessary to have sysimage.jl in the base folder as long as it's passed explicitly and current folder is base_dir. All these conditions are met.

@KristofferC
Copy link
Sponsor Member

Oh that is great then! Thanks!

@KristofferC KristofferC merged commit 60c0b52 into JuliaLang:master Mar 7, 2020
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