-
Notifications
You must be signed in to change notification settings - Fork 59
Mac wheel library linking issue #66
Comments
Hello, as far as I can tell, there have been no changes within As for building the macosx wheels, I created them by running the To improve the pip-install experience for Mac users, if having gcc installed would solve the issue, then perhaps we could more explicitly recommend |
Thanks for the quick response @jacksonllee. The issue is that currently I'm guessing it's looking for 9 because that's what you happened to have locally as well (and probably that's been the Homebrew version since the commit you linked to); I'm afraid I don't know enough to say how the explicit |
To confirm, I've just checked my Mac, and indeed it's on gcc 9.3. I also don't quite see how, if at all, the gcc version is hard-coded for building the wheels. One thing I can think of off the top of my head is that the Fortran compilation is done by the packaging tools imported from numpy which may be relevant to the issue at hand, but I'm not sure. We may also look at how other more established projects with Fortran code (scipy, etc.) build wheels and then we can emulate (the current set-up here is admittedly a couple years old and is probably due for maintenance). When our engineering resources allow it, we'll be able to look into this ticket more. |
I got similar error on my Mac. I only have gcc 10 but no 9. Basically, I install gcc 9 with brew, and link the installed folder to the one mentioned in the glmnet error:
|
Any chance the version could be bumped and new wheels made for gcc 10 (+ python 3.9)? |
For now, this solution work on mac (Not M1). Thanks ! |
Seems like the repo is kinda dead and no one is maintaining it! |
I've been using
glmnet==2.2.1
installed from Mac wheels withgcc==9.3.0
with no issues. But when my colleagues who didn't havegcc
installed yet tried to go through the setup, they ran into the following:Symlinking
/usr/local/opt/gcc/lib/gcc/10
to/usr/local/opt/gcc/lib/gcc/9
seems to do the trick but is obviously not ideal; is there maybe a change tosetup.py
or the wheel build script that would support both?The text was updated successfully, but these errors were encountered: