-
Notifications
You must be signed in to change notification settings - Fork 14
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
Set JULIA_CPU_TARGET
to make image portable
#185
Conversation
Some of the servers where we deploy the image doesn't support some of the instructions that the image is being built with. This means that the application need to be re-precompiled instead of loading the files created during image build. This patch sets `JULIA_CPU_TARGET` to the same value as the generic julia binaries use. In addition, this patch removes the permission relaxation of `/depot/compiled` which shouldn't be needed anymore. This was added in #14 to workaround a similar issue with stale precompilation files.
For posterity: The image built on bc5e987 is
but works on e.g.
The image built on e55ad18 is |
Some more debug output using JuliaLang/julia#50172:
|
I can't test why things worked on the latest build because the extra debug info is only available on Julia master. Fairly certain the image happened to be built with more generic instructions. At least with this PR things work as expected on the various CPUs that previously failed. |
Some of the servers where we deploy the image doesn't support some of the instructions that the image is being built with. This means that the application need to be re-precompiled instead of loading the files created during image build. This patch sets
JULIA_CPU_TARGET
to the same value as the generic julia binaries use.In addition, this patch removes the permission relaxation of
/depot/compiled
which shouldn't be needed anymore. This was added in #14 to workaround a similar issue with stale precompilation files.