-
Notifications
You must be signed in to change notification settings - Fork 720
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
fix LLVM easyconfigs as required by revamped custom easyblock for LLVM + add lit
and git
as build dependencies
#20902
Conversation
Co-authored-by: ocaisa <alan.ocais@cecam.org>
@boegelbot please test @ generoso |
@boegel: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2260799985 processed Message to humans: this is just bookkeeping information for me, |
@boegel |
|
@boegelbot please test @ generoso |
@branfosj: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2285756812 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2704250151 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Hmm, pretty consistent small group of tests that failed with bot:
Not sure what can cause these, they didn't pop up before. @Crivella I won't let this block the merge of easybuilders/easybuild-easyblocks#3373 + this PR, but we should maybe follow up on this? |
I will check on them tomorrow, but i do not think they are extremely relevant (seems to be only a problem with the tool llvm-mt) |
Test report by @boegel |
Test report by @boegel |
Test build for
That's entirely my bad, there is not OpenSSL 3 in that containers and I was reusing a bunch of modules... I'll upload a new test report for that one easyconfigs, and then this is good to go... \o/ |
lit
and git
as build dependencies
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extensively tested with easybuilders/easybuild-easyblocks#3373, so time to finally merge this.
@Crivella Thanks a lot for all your efforts on this, and sorry for the holdup...
Test report by @boegel |
…abled by default now
Test report by @boegel |
I think the failure in the CI was due to I've updated the dependency in 66b29d9 |
Is there a reason why we have both |
Umh, i think it is a leftover from the fact that i first build around 18.1.7 and than updated to 18.1.8. |
Test report by @boegel |
Test report by @boegel |
Going in, thanks @Crivella! |
EC files to compile LLVMcore (clang + flang-new + optional projects)
LLVM
easyblock for compilation of clang/flang + other llvm-projects easybuild-easyblocks#3373)More notes on the EB PR