-
Notifications
You must be signed in to change notification settings - Fork 15
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
GraalVM master tree fails to build Mandrel with JDK 22 EA on Windows #613
Comments
In my notes from 2023-10-30 jenkins:
IMHO still the same issue with unexported symbols? I recall seeing it fail on Linux for some time (better error message though) too and Linux seems fixed now. |
What is this file supposed to contain? I don't see it for Linux builds, unless I'm looking in the wrong place...
Care to clarify what that "unexported symbols" problem is/was? |
AFAIK it's Windows lib file specific, e.g
I don't have the build any more, but lemme re-run 5536ebd7 on Linux and see. |
I was able to compile this successfully on my system, which does use a different build script and does build on the C:\ drive. I know that that's not very helpful, but it's a data point that makes me wonder if something in the build harness has changed? |
Looking at the graal code, I would look at oracle#7557 which added this file. It's python, so simple to add prints and see that the drive letter is passed properly as part of the directory path. |
@stooke did that compile use mandrel-packaging? I'm starting to wonder if that file, Looking at oracle#7557 it's now a |
@jerboaa when I tried to use mandrel-packaging (you mean the Java-driven build?), I got an unknown package, something like org.graal.options (I don't remember the specifics). My current plan is to try to reconfigure my VM and build partially off the D drive.
that's why I asked about the D: drive; often Linux code ported to Windows gets that wrong. |
I think to reproduce, a mandrel-packaging style build would need to be used like GHA does:
|
Should be fixed with graalvm/mandrel-packaging#380. I.e. it was indeed a mandrel packaging issue. |
Closing as fixed. Windows Mandrel builds are ok again. |
Description
We see a Windows specific build failure of GraalVM master on Windows:
See: https://github.com/graalvm/mandrel/actions/runs/6819594271/job/18547716716#step:8:426
The same version worked fine to build on Linux:
https://github.com/graalvm/mandrel/actions/runs/6819594271/job/18547714537#step:8:614
The text was updated successfully, but these errors were encountered: