Skip to content

Conversation

@tanmoy1989
Copy link
Contributor

(created using eb --new-pr)

Copy link
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.

@tanmoy1989 didn't upudate any of the extensions relative to aiohttp-3.8.5-GCCcore-12.3.0.eb
it's preferred to update all extensions to the versions already used in the 2023b toolchain generation, or to the highest compatible version.

@tanmoy1989
Copy link
Contributor Author

@smoors: thanks for checking. I have now updated the extensions list with the latest available versions.

@smoors
Copy link
Contributor

smoors commented May 16, 2024

@boegelbot: please test @ generoso

@boegelbot
Copy link
Collaborator

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

PR test command 'EB_PR=20535 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs /opt/software/slurm/bin/sbatch --job-name test_PR_20535 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

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

Test report by @boegelbot
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
cns1 - Linux Rocky Linux 8.9, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/boegelbot/72242bddf92479d5191be331a18fb5a6 for a full test report.

@smoors
Copy link
Contributor

smoors commented May 16, 2024

looks like extension expandvars is missing:

File "/tmp/boegelbot/aiohttp/3.9.5/GCCcore-13.2.0/yarl/yarl-1.9.4/packaging/pep517_backend/_cython_configuration.py", line 10, in
from expandvars import expandvars
ModuleNotFoundError: No module named 'expandvars'

@laraPPr
Copy link
Contributor

laraPPr commented Jun 16, 2025

This version and toolchain have been merged with #20613. Closing this pr in favour of that one.

@laraPPr laraPPr closed this Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants