-
Notifications
You must be signed in to change notification settings - Fork 74
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
dlopen/dlcose change #397
Comments
Isn't BinDeps obsolescent?
|
BinDeps is still used by a lot of packages, it needs to be maintained, at least for a while. It looks like the build command is ran in a separate process, so maybe not closing the libraries isn't an issue. We could try it out. |
@jonathanBieler not closing libraries for cairo by using: daviehh/Cairo.jl@1f59845
|
Related: #396
also reported JuliaGraphics/Cairo.jl#271 and JuliaLang/julia#31517
According to the glib upstream https://gitlab.gnome.org/GNOME/pango/issues/363, dlopen followed by dlclose is not supported and it's surprising if it worked before.
Should changes be made to BinDeps.jl?
do not use
dlclose()
at all: maybe usingh = nothing
instead (is it safe? shall we also useGC.gc()
?)BinDeps.jl/src/dependencies.jl
Line 681 in f375ead
or, test if file exist at path; is so, use
addprocs()
andremotecall()
, thenrmprocs()
to execute in new process each time?The text was updated successfully, but these errors were encountered: