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

Restore #3344 and fix #5612 #5617

Merged
merged 2 commits into from
Jan 30, 2014
Merged

Restore #3344 and fix #5612 #5617

merged 2 commits into from
Jan 30, 2014

Conversation

ihnorton
Copy link
Member

I'll let someone else do the honors. In particular, there doesn't seem to be a transition strategy in #3344 other than "move yer stuff". Which is cool, but maybe it should be announced.

h/t @lendle for 2/3 of the fix for #5612/#5615.

@kmsquire @StefanKarpinski

jiahao added a commit that referenced this pull request Jan 30, 2014
@jiahao jiahao merged commit c49b0d3 into master Jan 30, 2014
@jiahao
Copy link
Member

jiahao commented Jan 30, 2014

Confirming that it fixes #5615.

@ihnorton ihnorton deleted the fix_5615 branch January 30, 2014 22:15
@kmsquire
Copy link
Member

In particular, there doesn't seem to be a transition strategy in #3344 other than "move yer stuff". Which is cool, but maybe it should be announced.

Thanks for point this out--I'll update NEWS, and make an announcement on julia-users and julia-dev.

Just to be clear, this change doesn't change or break existing package installations. Existing .julia directories should continue to work as-is.

@ihnorton
Copy link
Member Author

Ah, ok - I thought it would break for people on master.

@StefanKarpinski
Copy link
Member

No, but that's why I was nervous. We already supported finding ~/.julia/v1.2 directories, but now we create versioned package directories by default. If you've already got a setup without a version, nothing changes – or at least that's the theory.

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.

4 participants