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.

Sign up for GitHub

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

Jump to bottom

race in Make steps #45002

Closed
vtjnash opened this issue Apr 16, 2022 · 1 comment · Fixed by #53437
Closed

race in Make steps #45002

vtjnash opened this issue Apr 16, 2022 · 1 comment · Fixed by #53437
Labels
domain:building Build system, or building Julia or its dependencies

Comments

@vtjnash
Copy link
Sponsor Member

vtjnash commented Apr 16, 2022

if you do make all, the cli/loader_trampolines.o file may be attempted to be written by both the debug and release targets simultaneously, leading to this linker failure:

ld: file too small (length=0) file './loader_trampolines.o' for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[1]: *** [/Users/jameson/julia/usr/lib/libjulia-debug.1.9.dylib] Error 1
make: *** [julia-cli-debug] Error 2
@vtjnash vtjnash added the domain:building Build system, or building Julia or its dependencies label Apr 16, 2022
@ViralBShah
Copy link
Member

Dup of #34975

@vtjnash vtjnash mentioned this issue Feb 22, 2024
Merged
@vtjnash vtjnash reopened this Feb 22, 2024
fingolfin added a commit that referenced this issue Feb 23, 2024
@fingolfin
Add debug variant of loader_trampolines.o (#53437)
fee198b 
This prevents a race condition when building 'julia-cli-debug
julia-cli-release' simultaneously (as we do for libjulia_jll, and also
generally seems appropriate given what is done for all other source
files.

Motivated by JuliaPackaging/Yggdrasil#8151 so
I'll first see if it works there.

Closes #45002.
KristofferC pushed a commit that referenced this issue Feb 26, 2024
@fingolfin @KristofferC
Add debug variant of loader_trampolines.o (#53437)
baa56c5 
This prevents a race condition when building 'julia-cli-debug
julia-cli-release' simultaneously (as we do for libjulia_jll, and also
generally seems appropriate given what is done for all other source
files.

Motivated by JuliaPackaging/Yggdrasil#8151 so
I'll first see if it works there.

Closes #45002.

(cherry picked from commit fee198b)
KristofferC pushed a commit that referenced this issue Feb 26, 2024
@fingolfin @KristofferC
Add debug variant of loader_trampolines.o (#53437)
63ebed7 
This prevents a race condition when building 'julia-cli-debug
julia-cli-release' simultaneously (as we do for libjulia_jll, and also
generally seems appropriate given what is done for all other source
files.

Motivated by JuliaPackaging/Yggdrasil#8151 so
I'll first see if it works there.

Closes #45002.

(cherry picked from commit fee198b)
KristofferC pushed a commit that referenced this issue Feb 26, 2024
@fingolfin @KristofferC
Add debug variant of loader_trampolines.o (#53437)
5d971b9 
This prevents a race condition when building 'julia-cli-debug
julia-cli-release' simultaneously (as we do for libjulia_jll, and also
generally seems appropriate given what is done for all other source
files.

Motivated by JuliaPackaging/Yggdrasil#8151 so
I'll first see if it works there.

Closes #45002.

(cherry picked from commit fee198b)
tecosaur pushed a commit to tecosaur/julia that referenced this issue Mar 4, 2024
@fingolfin @tecosaur
Add debug variant of loader_trampolines.o (JuliaLang#53437)
ae63cbe 
This prevents a race condition when building 'julia-cli-debug
julia-cli-release' simultaneously (as we do for libjulia_jll, and also
generally seems appropriate given what is done for all other source
files.

Motivated by JuliaPackaging/Yggdrasil#8151 so
I'll first see if it works there.

Closes JuliaLang#45002.
mkitti pushed a commit to mkitti/julia that referenced this issue Mar 7, 2024
@fingolfin @mkitti
Add debug variant of loader_trampolines.o (JuliaLang#53437)
88fb12a 
This prevents a race condition when building 'julia-cli-debug
julia-cli-release' simultaneously (as we do for libjulia_jll, and also
generally seems appropriate given what is done for all other source
files.

Motivated by JuliaPackaging/Yggdrasil#8151 so
I'll first see if it works there.

Closes JuliaLang#45002.
Drvi pushed a commit to RelationalAI/julia that referenced this issue Jun 7, 2024
@fingolfin @Drvi
Add debug variant of loader_trampolines.o (JuliaLang#53437)
9069509 
This prevents a race condition when building 'julia-cli-debug
julia-cli-release' simultaneously (as we do for libjulia_jll, and also
generally seems appropriate given what is done for all other source
files.

Motivated by JuliaPackaging/Yggdrasil#8151 so
I'll first see if it works there.

Closes JuliaLang#45002.

(cherry picked from commit fee198b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
domain:building Build system, or building Julia or its dependencies
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add debug variant of loader_trampolines.o fingolfin/julia
2 participants
@vtjnash @ViralBShah