-
Notifications
You must be signed in to change notification settings - Fork 107
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
kclvm-runner build error. #323
Comments
Of course, you can open a PR to fix it directly. Thank you for your feedback. @my-vegetable-has-exploded |
I reran ci for the current main branch, but build-and-test-ubuntu failed. And I found the kclvm-builder container had been update 3days algo, and the version of lld/Common/Driver.h in the container maybe 14.x or 15.x not 12.x. I think that's maybe why this ci failed. @Peefy |
@my-vegetable-has-exploded I have adjusted the LLVM version in the image to 13, and you can rerun CI. |
I found it still fails. More details can be seen in these two ci build-and-test-ubuntu build-and-test-ubuntu |
@my-vegetable-has-exploded https://github.com/KusionStack/KCLVM/actions/runs/3616626847/jobs/6095961037 The main branch CI is normal, now U can open your PR please. |
Sorry , I can't reproduce the bug again now, there may be something wrong with my environment. Sorry for taking up your time. |
Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)
environment:
NAME="Ubuntu"
VERSION="20.04.3 LTS (Focal Fossa)"
execute
cargo test --package kclvm-tools
2. What did you expect to see? (Required)
3. What did you see instead (Required)
4. What is your KusionStack components version? (Required)
main branch
I found this line doesn't work in my environment.
println!("cargo:libdir={}", libdir);
/root/KCLVM/kclvm/runner/build.rs
when i change it to
println!("cargo:rustc-link-search={}", libdir);
I can build kclvm-runner successfully.
If needed, i can open a pr to solve it.
The text was updated successfully, but these errors were encountered: