Skip to content

add patch to fix linking error for imread extension in scikit-image easyconfigs using intel toolchain#15054

Merged
smoors merged 1 commit intoeasybuilders:developfrom
boegel:20220225213343_new_pr_scikit-image0162
Feb 26, 2022
Merged

add patch to fix linking error for imread extension in scikit-image easyconfigs using intel toolchain#15054
smoors merged 1 commit intoeasybuilders:developfrom
boegel:20220225213343_new_pr_scikit-image0162

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Feb 25, 2022

(created using eb --new-pr)
fixes #14219

@boegel boegel added the bug fix label Feb 25, 2022
@boegel boegel added this to the next release (4.5.4?) milestone Feb 25, 2022
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 25, 2022

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node3103.skitty.os - Linux CentOS Linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/b4209eed88d40314702e8251a055b4f2 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 25, 2022

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Feb 25, 2022

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

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

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1051265871 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).

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 25, 2022

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node3524.doduo.os - Linux RHEL 8.4, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/c1abb3984712343dbcca8c2f2ae9919a for a full test report.

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Feb 25, 2022

Test report by @boegelbot
SUCCESS
Build succeeded for 4 out of 4 (2 easyconfigs in total)
cns3 - 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/353592a2a14dacab65714efc56b08ee5 for a full test report.

smoors
smoors approved these changes Feb 26, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

lgtm

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Feb 26, 2022

Going in, thanks @boegel!

@smoors smoors merged commit 136b15f into easybuilders:develop Feb 26, 2022
@zdemat zdemat mentioned this pull request Feb 26, 2022
Merged
@boegel boegel deleted the 20220225213343_new_pr_scikit-image0162 branch February 26, 2022 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@smoors smoors smoors approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.5.4

Development

Successfully merging this pull request may close these issues.

"multiple definition of" linking error when building scikit-image with intel toolchain

3 participants

@boegel @boegelbot @smoors