Skip to content

update sanity check in wxPython easyblock for version 4.2.0#3023

Merged
boegel merged 1 commit intoeasybuilders:developfrom
sassy-crick:20231023210126_new_pr_wxpython
Oct 27, 2023
Merged

update sanity check in wxPython easyblock for version 4.2.0#3023
boegel merged 1 commit intoeasybuilders:developfrom
sassy-crick:20231023210126_new_pr_wxpython

Conversation

@sassy-crick
Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

@boegel boegel changed the title Sanity-test expanded to 4.2.0 version update sanity check in wxPython easyblock for version 4.2.0 Oct 25, 2023
@boegel boegel added this to the next release (4.8.2?) milestone Oct 25, 2023
@boegel boegel added the update label Oct 25, 2023
@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 25, 2023

@boegelbot please test @ jsc-zen2
EB_ARGS="wxPython-4.1.1-foss-2021a.eb wxPython-4.2.0-foss-2021b.eb wxPython-4.2.1-foss-2022a.eb"

@boegelbot
Copy link
Copy Markdown

@boegel: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster

PR test command 'EB_PR=3023 EB_ARGS="wxPython-4.1.1-foss-2021a.eb wxPython-4.2.0-foss-2021b.eb wxPython-4.2.1-foss-2022a.eb" EB_REPO=easybuild-easyblocks /opt/software/slurm/bin/sbatch --mem-per-cpu=4000M --job-name test_PR_3023 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen2.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1778767123 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 wxPython-4.1.1-foss-2021a.eb
  • SUCCESS wxPython-4.2.1-foss-2022a.eb
  • SUCCESS wxWidgets-3.2.0-GCC-11.2.0.eb
  • SUCCESS dbus-glib-0.112-GCCcore-11.2.0.eb
  • SUCCESS GConf-3.2.6-GCCcore-11.2.0.eb
  • SUCCESS attrdict3-2.0.2-GCCcore-11.2.0.eb
  • SUCCESS wxPython-4.2.0-foss-2021b.eb

Build succeeded for 7 out of 7 (3 easyconfigs in total)
jsczen2c1.int.jsc-zen2.easybuild-test.cluster - Linux Rocky Linux 8.5, x86_64, AMD EPYC 7742 64-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/boegelbot/565c2fe72ce2833842bb3622e1bdec0c for a full test report.

@bartoldeman bartoldeman self-requested a review October 25, 2023 16:00
@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 27, 2023

This only really matters for wxPython easyconfigs for v4.2.0 or newer that do not include wxWidgets as dependency (since only then is majver actually used (which explains why easybuilders/easybuild-easyconfigs#16856 could get merged without this).

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 merged commit 36ecfa5 into easybuilders:develop Oct 27, 2023
@sassy-crick sassy-crick deleted the 20231023210126_new_pr_wxpython branch November 3, 2023 16:50
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.

3 participants