Skip to content

use clean_dir instead of remove_dir on installation directory is custom easyblock for Tkinter#3929

Merged
boegel merged 1 commit intoeasybuilders:developfrom
smoors:20250915153740_new_pr_tkinter
Apr 8, 2026
Merged

use clean_dir instead of remove_dir on installation directory is custom easyblock for Tkinter#3929
boegel merged 1 commit intoeasybuilders:developfrom
smoors:20250915153740_new_pr_tkinter

Conversation

@smoors
Copy link
Copy Markdown
Contributor

@smoors smoors commented Sep 15, 2025

@smoors
Copy link
Copy Markdown
Contributor Author

smoors commented Sep 17, 2025

Test report by @smoors

Overview of tested easyconfigs (in order)

  • SUCCESS Tkinter-3.12.3-GCCcore-13.3.0.eb

Build succeeded for 1 out of 1 (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/c68130a4cdde4bac18de300b633543cc for a full test report.

@smoors
Copy link
Copy Markdown
Contributor Author

smoors commented Sep 17, 2025

Test report by @smoors

Overview of tested easyconfigs (in order)

  • SUCCESS Tkinter-3.11.3-GCCcore-12.3.0.eb

Build succeeded for 1 out of 1 (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/c09879b93912e42f7fb3421bc8ce3b60 for a full test report.

@boegel boegel changed the title use clean_dir instead of remove_dir on installation directory for Tkinter easyblock use clean_dir instead of remove_dir on installation directory for Tkinter easyblock Mar 19, 2026
@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 19, 2026

@boegelbot please test @ jsc-zen3
EB_ARGS="--installpath /tmp/boegelbot/pr3929 Tkinter-2.7.18-GCCcore-11.2.0.eb Tkinter-3.10.8-GCCcore-12.2.0.eb Tkinter-3.13.5-GCCcore-14.3.0.eb"

@boegel boegel added this to the next release (5.2.2?) milestone Mar 19, 2026
@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=3929 EB_ARGS="--installpath /tmp/boegelbot/pr3929 Tkinter-2.7.18-GCCcore-11.2.0.eb Tkinter-3.10.8-GCCcore-12.2.0.eb Tkinter-3.13.5-GCCcore-14.3.0.eb" EB_CONTAINER= EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_3929 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

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

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 changed the title use clean_dir instead of remove_dir on installation directory for Tkinter easyblock use clean_dir instead of remove_dir on installation directory is custom easyblock for Tkinter Mar 19, 2026
@boegelbot
Copy link
Copy Markdown

Test report by @boegelbot

Overview of tested easyconfigs (in order)

  • SUCCESS Tkinter-2.7.18-GCCcore-11.2.0.eb

  • SUCCESS Tkinter-3.10.8-GCCcore-12.2.0.eb

  • SUCCESS Tkinter-3.13.5-GCCcore-14.3.0.eb

Build succeeded for 3 out of 3 (total: 28 mins 26 secs) (3 easyconfigs in total)
jsczen3c1.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/407cfbbf3e8d50b3952ba6ab84d762ac for a full test report.

@boegel boegel merged commit 351532f 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