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

julia-1.7.1 with curl >=7.81.0: Build failure #44657

Closed
ChristianS99 opened this issue Mar 17, 2022 · 12 comments
Closed

julia-1.7.1 with curl >=7.81.0: Build failure #44657

ChristianS99 opened this issue Mar 17, 2022 · 12 comments
Labels
building Build system, or building Julia or its dependencies

Comments

@ChristianS99
Copy link

Generating REPL precompile statements... 26/36ERROR: LoadError: IOError: stream is closed or unusable
Stacktrace:
 [1] check_open
   @ ./stream.jl:386 [inlined]
 [2] uv_write_async(s::Base.TTY, p::Ptr{UInt8}, n::UInt64)
   @ Base ./stream.jl:1018
 [3] uv_write(s::Base.TTY, p::Ptr{UInt8}, n::UInt64)
   @ Base ./stream.jl:981
 [4] unsafe_write(s::Base.TTY, p::Ptr{UInt8}, n::UInt64)
   @ Base ./stream.jl:1064
 [5] mktemp(fn::Main.anonymous.var"#2#6"{Set{String}, String, Base.DevNull}, parent::String)
   @ Base.Filesystem ./file.jl:722
in expression starting at /var/tmp/portage/dev-lang/julia-1.7.1-r3/work/julia-1.7.1/contrib/generate_precompile.jl:3

with curl-7.80.0 build is fine

@ChristianS99
Copy link
Author

curl-7.82.0: can't build, too, but the build just doesn't continue
the last output is:
Generating REPL precompile statements... 25/36
there are two julia processes:
/var/tmp/portage/dev-lang/julia-1.7.1-r3/work/julia-1.7.1/usr/bin/julia -O3 -C native --output-o /var/tmp/portage/dev-lang/julia-1.7.1-r3/work/julia-1.7.1/usr/lib/julia/sys-o.a.tmp --startup-file=no --warn-overwrite=yes --sysimage /var/tmp/portage/dev-lang/julia-1.7.1-r3/work/julia-1.7.1/usr/lib/julia/sys.ji /var/tmp/portage/dev-lang/julia-1.7.1-r3/work/julia-1.7.1/contrib/generate_precompile.jl 1
and
/var/tmp/portage/dev-lang/julia-1.7.1-r3/work/julia-1.7.1/usr/bin/julia -O0 --trace-compile=/var/tmp/portage/dev-lang/julia-1.7.1-r3/temp/jl_PoLhG9 --sysimage /var/tmp/portage/dev-lang/julia-1.7.1-r3/work/julia-1.7.1/usr/lib/julia/sys.ji --cpu-target=native --startup-file=no --color=yes -e import REPL; REPL.Terminals.is_precompiling[] = true -i --color=yes -e import REPL; REPL.Terminals.is_precompiling[] = true

added the compile trace:
jl_PoLhG9.log

Mentioning here, guess it's related

@ChristianS99 ChristianS99 changed the title julia-1.7.1 with curl-7.81.0: Build failure julia-1.7.1 with curl >7.81.0: Build failure Mar 17, 2022
@ChristianS99 ChristianS99 changed the title julia-1.7.1 with curl >7.81.0: Build failure julia-1.7.1 with curl >=7.81.0: Build failure Mar 17, 2022
@ViralBShah ViralBShah added building Build system, or building Julia or its dependencies system:linux Affects only Linux labels Mar 17, 2022
@giordano
Copy link
Contributor

Sounds much like a duplicate of #43249

@ChristianS99
Copy link
Author

Looks similar, but I tried with proposed patches or older versions of libgit2, but only installing an old version of curl helped.
Therefore I made a new issue.

@ngam
Copy link

ngam commented Mar 19, 2022

at conda-forge, we had a similar problem and it was resolved with updating the downloads module.

https://github.com/conda-forge/julia-feedstock/pull/178/files

Basically, get the latest Downloads from master or something, which should already fix things. However, we also had patches for libgit2, etc. so you may need those as well.

@ngam
Copy link

ngam commented Mar 19, 2022

(disclaimer: if I remember correctly, I thought we were getting segfaults at the time, though I am not too sure)

@mtasaka
Copy link

mtasaka commented Mar 19, 2022

This is actually julia is segfaulting with curl 7.81.0
https://bugzilla.redhat.com/show_bug.cgi?id=2056842
https://bugzilla-attachments.redhat.com/attachment.cgi?id=1862510

So my guess is that in fact #43249 fixed nothing.

@mtasaka
Copy link

mtasaka commented Mar 19, 2022

This should be fixed with JuliaLang/Downloads.jl#164

@giordano
Copy link
Contributor

Nothing to do here then, right?

@giordano giordano removed the system:linux Affects only Linux label Mar 19, 2022
@ngam
Copy link

ngam commented Mar 19, 2022

Nothing to do here then, right?

The only thing would be be backport a higher downloads version to 1.7.1 and prior, and that's it...

DOWNLOADS_BRANCH = release-1.5

but I don't know if that's actually logical --- I get there is an LTS support thing going on, but that won't be 1.7.1, right? It would be 1.7.2/1.6.5 and whatever comes after them. So, yes, I would agree nothing to do here besides pointing out the fix if someone really really really wants to use 1.7.1 at this point in time...

@ngam
Copy link

ngam commented Mar 19, 2022

i.e. it would make more sense to upgrade julia if one wants to use the upgraded curl imo

@giordano
Copy link
Contributor

The only thing would be be backport a higher downloads version to 1.7.1 and prior, and that's it...

It's quite hard to backport changes to an already released version. However the fix in Downloads.jl is already in the master branch of Julia and it has been also backported to the 1.7 branch with #44433.

I don't see anything else we can possibly do here, so I'm going to close this issue.

@ChristianS99
Copy link
Author

OK, thx. When it works in 1.7.2, that's fine for me. Thx for clarification

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies
Projects
None yet
Development

No branches or pull requests

5 participants