Skip to content

disable use of -Werror in recent NSS easyconfigs#16571

Merged
jfgrimm merged 1 commit intoeasybuilders:developfrom
boegel:20221107165537_new_pr_NSS365
Nov 8, 2022
Merged

disable use of -Werror in recent NSS easyconfigs#16571
jfgrimm merged 1 commit intoeasybuilders:developfrom
boegel:20221107165537_new_pr_NSS365

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Nov 7, 2022

(created using eb --new-pr)

@boegel boegel added the bug fix label Nov 7, 2022
@boegel boegel added this to the next release (4.6.3?) milestone Nov 7, 2022
@boegel boegel added the EESSI Related to EESSI project label Nov 7, 2022
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Nov 7, 2022

Test report by @boegel
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
node3135.skitty.os - Linux RHEL 8.4, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/a102af5b8fa571ddb3aeeabc952eb30f for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Nov 7, 2022

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Nov 7, 2022

@boegel: Request for testing this PR well received on login1

PR test command 'EB_PR=16571 EB_ARGS= EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_16571 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 9453

Test results coming soon (I hope)...

Details

- notification for comment with ID 1306070823 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Nov 7, 2022

Test report by @boegelbot
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
cns2 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/1ad0485e19bcc9860875d7a8d9929317 for a full test report.

@jfgrimm
Copy link
Copy Markdown
Member

jfgrimm commented Nov 8, 2022

Test report by @jfgrimm
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
node146.pri.viking.alces.network - Linux CentOS Linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6138 CPU @ 2.00GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/1ab48e57aeee623d960a8a534ac48f8b for a full test report.

jfgrimm
jfgrimm approved these changes Nov 8, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@jfgrimm jfgrimm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jfgrimm
Copy link
Copy Markdown
Member

jfgrimm commented Nov 8, 2022

Going in, thanks @boegel!

@jfgrimm jfgrimm merged commit 6af3441 into easybuilders:develop Nov 8, 2022
@boegel boegel deleted the 20221107165537_new_pr_NSS365 branch November 8, 2022 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jfgrimm jfgrimm jfgrimm approved these changes

Assignees

No one assigned

Labels

bug fix EESSI Related to EESSI project

Projects

None yet

Milestone

4.7.0

Development

Successfully merging this pull request may close these issues.

3 participants

@boegel @boegelbot @jfgrimm