-
Notifications
You must be signed in to change notification settings - Fork 3
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
[CI] Test with Poplar 3.3 #45
base: main
Are you sure you want to change the base?
Conversation
157ea0e
to
7aa4f39
Compare
With #62 we now have a version of Clang.jl compatible with Julia v1.11, but building the bindings is failing because the Poplar header files can't be found. The problem is that instead of explicitly setting the environment variables like For what is worth, building of the package works locally for me inside
so there's hope this will eventually work, but the problem is that here we aren't going through the exact same path as we aren't using the bash entrypoint. |
ecd2d8e
to
0728e50
Compare
I believe there's a bug in
while it works fine in #48 with the same version of |
The tricky part is that we need to upgrade both polar (3.2->3.3) and the wrapper generator script, probably CxxWrap as well. |
Yes, each upgrade of this package is tricky 🥲 CxxWrap was upgraded in #66, I probably need to rebase this branch on main |
0728e50
to
9f03bd1
Compare
Clang.jl is currently not compatible with Julia v1.11, so we can't test this at the moment. Fixing #18 would let us reduce the number of dependencies, including Clang.jl itself (it's only needed to generate the wrappers, not at runtime).