-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Build failure in OpenBLAS #44517
Comments
I presume Line 905 in f731c38
|
Even if the
The Julia makefile passes |
@fxcoudert I understand your frustration and much appreciate your willingness to help, but I'd just like to ask you to bear in mind that all the problems you're facing are because the build from source is barely tested, if at all, by developers and users, as you probably well figured out by now. Apple Silicon in particular is even more problematic because only a handful of developers have access to this system. We prefer instead to reuse the binaries built in Yggdrasil precisely to avoid everybody, users and developers, to go through all the hoops of supporting building complicated libraries on dozens of different platforms and even more so configurations. With BinaryBuilder, instead, we build only once in a single controlled environment for all supported platforms, and then no one else needs to fight with stubborn build systems, niche operating systems, disappearing versions of python and such. [Side note: it isn't advertised anywhere because we don't have strong guarantees, but in practice builds with BinaryBuilder are usually reproducible and verifiable independently by other users using the same tool, which at least should relieve people worried about what nasty bits we may put into those prebuilt binaries.] This also sensibly cuts down CI time for the project, as we don't need to rebuild large binary dependencies like LLVM for every single commit pushed to the Supporting a full build from source would be great, but we don't have enough resources to keep up with CI for that code path, which hopefully explains all of your issues and why no one seems to be worried (which isn't the case!). We'll eventually sort all these problems out, but it'll take some time. I'm not even that familiar myself with Julia's build system (and I'm quite stretched with my non-Julia job until next week), I'm learning with you 🙂 |
@giordano I do understand the current state of things, and I thank you for your help in my various bug reports. I'd just like to point out that there are many users (distros, etc) for which installing binaries compiled on a black-box build system somewhere is not acceptable. The nice thing about open source software is the possibility for users to build from source, tune, or even if they install binaries know that they come from a trusted source, independent from individual package authors. That's why I find it surprising that build-from-source is considered a second-class citizen, and I hope that by raising issues and proposing patches (when I can) I can help — I am not even a Julia user myself, despite investing a significant amount of time to try and get it in buildable shape. I know as a maintainer of my own open source stuff how useful it is to have well-documented, reproducible bug reports with debug info, so I try to provide those in the clearest fashion. |
For this specific issue, I think it was introduced by #42538 |
@DilumAluthge it may perhaps be nice to test the full source only build every night on CI. Maybe just Linux may be a good start. Is such a thing possible? |
@fxcoudert Broadly we do agree with you, and are happy for your contributions. As @giordano explained, it is a matter of dev time but we would love to have non BB be more robust. M1 is just also too new and it will take some time to shake issues out on that architecture. |
I understand, but note that most of the issues raised occur on macOS Intel as well. |
We actually already do this. Once per day, we build and test Julia with |
The
Looking at the other dependencies and tools, it looks like
Does that seem right? |
This may be a @staticfloat question. |
I've opened a PR for the objconv part: #45391 |
Default build of OpenBLAS on macOS ARM (Apple Silicon) within Julia:
gives the following build error:
I tried passing
OPENBLAS_TARGET_ARCH=VORTEX
tomake
as an additional parameter, which is passed down to openblas asmake TARGET=VORTEX
. But that does not appear to solve the issue.The text was updated successfully, but these errors were encountered: