Skip to content

remove trailing slash in SLEPC_DIR in configure step of custom easyblock for SLEPc#3229

Merged
boegel merged 1 commit intoeasybuilders:developfrom
furstj:develop
Mar 15, 2024
Merged

remove trailing slash in SLEPC_DIR in configure step of custom easyblock for SLEPc#3229
boegel merged 1 commit intoeasybuilders:developfrom
furstj:develop

Conversation

@furstj
Copy link
Copy Markdown
Contributor

@furstj furstj commented Feb 15, 2024

Compilation of SLEPc 3.20.1 (easyconfig pr #19863) failed because of a trailing slash in SLEPC_DIR. See similar issue for PETSc resolved in easyblock pr #3086

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel added the bug fix label Feb 28, 2024
@boegel boegel added this to the release after 4.9.0 milestone Feb 28, 2024
@boegel boegel changed the title Remove trailing slash in SLEPC_DIR Remove trailing slash in SLEPC_DIR in configure step of custom easyblock for SLEPc Feb 28, 2024
@boegel boegel changed the title Remove trailing slash in SLEPC_DIR in configure step of custom easyblock for SLEPc remove trailing slash in SLEPC_DIR in configure step of custom easyblock for SLEPc Feb 28, 2024
@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 15, 2024

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS SLEPc-3.12.2-foss-2019b-Python-3.7.4.eb
  • SUCCESS SLEPc-3.15.1-foss-2021a.eb
  • SUCCESS SLEPc-3.17.2-foss-2022a.eb
  • SUCCESS SLEPc-3.18.2-intel-2021b.eb

Build succeeded for 4 out of 4 (4 easyconfigs in total)
node3136.skitty.os - Linux RHEL 8.8, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/boegel/fd461118fe8bf452244bf6ae25877cc9 for a full test report.

@boegel boegel merged commit e46f025 into easybuilders:develop Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants