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

Fix 7zip path for julia 1.3 #407

Merged
merged 1 commit into from
Dec 5, 2019
Merged

Conversation

oxinabox
Copy link
Contributor

Since it got moved

if isdefined(Base, :LIBEXECDIR)
const exe7z = joinpath(Sys.BINDIR, Base.LIBEXECDIR, "7z.exe")
else
const exe7z = joinpath(Sys.BINDIR, "7z.exe")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
const exe7z = joinpath(Sys.BINDIR, "7z.exe")
const exe7z = joinpath(Compat.Sys.BINDIR, "7z.exe")

@JackDunnNZ
Copy link

Any chance this can be merged? It seems to be breaking ScikitLearn.jl for me

@musm
Copy link
Contributor

musm commented Dec 4, 2019

bump yes this is also causing failures

@musm
Copy link
Contributor

musm commented Dec 4, 2019

Can we merge this ASAP and tag a release? It's causing any package using BinDeps failure on 1.3

@musm
Copy link
Contributor

musm commented Dec 5, 2019

@ViralBShah I'm not sure who has permissions here, but this is a pretty important PR to get merge and make a release. I'ts has broken Blosc which has broken CMake and by extension HDF5.jl

@ViralBShah ViralBShah merged commit 0f52a01 into JuliaPackaging:master Dec 5, 2019
@ViralBShah
Copy link
Member

@musm gave you commit access here.

@musm
Copy link
Contributor

musm commented Dec 5, 2019

Thank you very much!

@musm
Copy link
Contributor

musm commented Dec 5, 2019

Any chance we can install https://github.com/JuliaRegistries/Registrator.jl here? It can only be performed by organization owners.

We'll also need to get in #408

@ViralBShah
Copy link
Member

@musm I have temporarily invited you to be owner of JuliaPackaging, to help smoothen out things here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants