Skip to content

use clean_dir instead of remove_dir on installation directory in custom easyblock for GATE#3932

Merged
boegel merged 1 commit intoeasybuilders:developfrom
smoors:20250915153948_new_pr_gate
Apr 8, 2026
Merged

use clean_dir instead of remove_dir on installation directory in custom easyblock for GATE#3932
boegel merged 1 commit intoeasybuilders:developfrom
smoors:20250915153948_new_pr_gate

Conversation

@smoors
Copy link
Copy Markdown
Contributor

@smoors smoors commented Sep 15, 2025

@smoors
Copy link
Copy Markdown
Contributor Author

smoors commented Sep 21, 2025

Test report by @smoors

Overview of tested easyconfigs (in order)

  • SUCCESS CLHEP-2.4.7.1-GCC-12.3.0.eb
  • SUCCESS Geant4-data-11.2.eb
  • SUCCESS Geant4-11.2.2-GCC-12.3.0.eb
  • SUCCESS GATE-9.4-foss-2023a.eb

Build succeeded for 4 out of 4 (1 easyconfigs in total)
node600.hydra.os - Linux Rocky Linux 9.6, x86_64, AMD EPYC 9384X 32-Core Processor (zen4), Python 3.9.21
See https://gist.github.com/smoors/40a9bb4ac0c099293ee1f7b52b30f19a for a full test report.

@boegel boegel changed the title use clean_dir instead of remove_dir on installation directory for GATE easyblock use clean_dir instead of remove_dir on installation directory for GATE easyblock Mar 19, 2026
@boegel boegel added this to the next release (5.2.2?) milestone Mar 19, 2026
@boegel boegel changed the title use clean_dir instead of remove_dir on installation directory for GATE easyblock use clean_dir instead of remove_dir on installation directory in custom easyblock for GATE Mar 19, 2026
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 (especially since changes are in dead code)

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 19, 2026

@boegelbot please test @ jsc-zen3
CORE_CNT=16
EB_ARGS="--installpath /tmp/boegelbot/pr3932 GATE-10.0.2-foss-2023b.eb"

@boegelbot
Copy link
Copy Markdown

@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=3932 EB_ARGS="--installpath /tmp/boegelbot/pr3932 GATE-10.0.2-foss-2023b.eb" EB_CONTAINER= EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_3932 --ntasks="16" ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 4090248655 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

Test report by @boegelbot

Overview of tested easyconfigs (in order)

  • SUCCESS GATE-10.0.2-foss-2023b.eb

Build succeeded for 1 out of 1 (total: 28 mins 33 secs) (1 easyconfigs in total)
jsczen3c3.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.7, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.25
See https://gist.github.com/boegelbot/38c2ac6fd3e75eaed4a07c2576444d1c for a full test report.

@boegel boegel merged commit 4e97d05 into easybuilders:develop Apr 8, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants