Skip to content

Put a try-guard around this import for newer pythons#334

Merged
smoors merged 2 commits into
EESSI:mainfrom
casparvl:add_import_guard
May 6, 2026
Merged

Put a try-guard around this import for newer pythons#334
smoors merged 2 commits into
EESSI:mainfrom
casparvl:add_import_guard

Conversation

@casparvl
Copy link
Copy Markdown
Collaborator

@casparvl casparvl commented May 5, 2026

I ran into issues on a system which had python 3.12, which lacks pkg_resources. Essentially, you can't import anything from the eessi namespace there

@casparvl
Copy link
Copy Markdown
Collaborator Author

casparvl commented May 5, 2026

I can confirm that this resolved my issue:

EESSI_TESTSUITE_URL=https://github.com/casparvl/test-suite.git EESSI_TESTSUITE_BRANCH=add_import_guard EESSI_CONFIGS_TESTSUITE_URL=https://github.com/EESSI/test-suite.git EESSI_CONFIGS_TESTSUITE_BRANCH=main EESSI_CI_SYSTEM_NAME=bsc_hca ${HOME}/EESSI_CI/CI/run_reframe.sh

Now listed all tests succesfully.

Copy link
Copy Markdown
Collaborator

@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.

lgtm

@smoors smoors merged commit 20469dc into EESSI:main May 6, 2026
16 checks passed
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.

2 participants