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

precompile loads the wrong library after recompile_stale #18069

Closed
vtjnash opened this issue Aug 16, 2016 · 6 comments · Fixed by #18150
Closed

precompile loads the wrong library after recompile_stale #18069

vtjnash opened this issue Aug 16, 2016 · 6 comments · Fixed by #18150
Assignees
Labels
bug Indicates an unexpected problem or unintended behavior compiler:precompilation Precompilation of modules priority This should be addressed urgently
Milestone

Comments

@vtjnash
Copy link
Member

vtjnash commented Aug 16, 2016

the return value of compilecache is being ignored by recompile_stale, leading to state corruption and loading of the wrong file:

juser@juliabox:~$ /opt/julia-0.4.6/bin/julia                                                                                                                           
               _
   _       _ _(_)_     |  A fresh approach to technical computing                                                                                                      
  (_)     | (_) (_)    |  Documentation: http://docs.julialang.org                                                                                                     
   _ _   _| |_  __ _   |  Type "?help" for help.                                                                                                                       
  | | | | | | |/ _` |  |                                                                                                                                               
  | | |_| | | | (_| |  |  Version 0.4.6 (2016-06-19 17:16 UTC)                                                                                                         
 _/ |\__'_|_|_|\__'_|  |  Official http://julialang.org/ release                                                                                                       
|__/                   |  x86_64-unknown-linux-gnu                                                                                                                     

julia> using JSON                                                                                                                                                      
INFO: Recompiling stale cache file /opt/julia_packages/.julia/lib/v0.4/JSON.ji for module JSON.                                                                        

julia> Base.module_uuid(JSON)                                                                                                                                          
0x0000000000096afb                                                                                                                                                     

julia>                                                                                                                                                                 
juser@juliabox:~$ /opt/julia-0.4.6/bin/julia                                                                                                                           
               _
   _       _ _(_)_     |  A fresh approach to technical computing                                                                                                      
  (_)     | (_) (_)    |  Documentation: http://docs.julialang.org                                                                                                     
   _ _   _| |_  __ _   |  Type "?help" for help.                                                                                                                       
  | | | | | | |/ _` |  |                                                                                                                                               
  | | |_| | | | (_| |  |  Version 0.4.6 (2016-06-19 17:16 UTC)                                                                                                         
 _/ |\__'_|_|_|\__'_|  |  Official http://julialang.org/ release                                                                                                       
|__/                   |  x86_64-unknown-linux-gnu                                                                                                                     

julia> using JSON                                                                                                                                                      

julia> Base.module_uuid(JSON)                                                                                                                                          
0x000000001644d120                                                                                                                                                     
@vtjnash vtjnash added bug Indicates an unexpected problem or unintended behavior priority This should be addressed urgently labels Aug 16, 2016
@JeffBezanson
Copy link
Member

How did you trigger this? The uuids match for me. Or is it 0.4 only?

@ViralBShah
Copy link
Member

It is easily triggered on JuliaBox for 0.4.6 when trying to use pyplot as a new user.

@vtjnash
Copy link
Member Author

vtjnash commented Aug 17, 2016

Yes, tested this from JuliaBox and see the issue. I think I'll do the minimal fix for 0.4 and work on a more comprehensive fix for 0.5. Apparently I need to dog-food the non-trivial precompile states more often.

@vtjnash vtjnash self-assigned this Aug 17, 2016
@kshyatt kshyatt added the compiler:precompilation Precompilation of modules label Aug 17, 2016
@ViralBShah
Copy link
Member

The JuliaBox.com migration is also blocked on this, which we are trying to do this weekend.

@ViralBShah
Copy link
Member

Cc @tanmaykm @nkottary

@ViralBShah
Copy link
Member

Is this a 0.5 blocker?

@ViralBShah ViralBShah added this to the 0.5.x milestone Aug 19, 2016
tkelman pushed a commit that referenced this issue Aug 29, 2016
vtjnash added a commit that referenced this issue Aug 30, 2016
mfasi pushed a commit to mfasi/julia that referenced this issue Sep 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior compiler:precompilation Precompilation of modules priority This should be addressed urgently
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants