Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Explain search for sub-toolchains in robot-paths #258

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Flamefire
Copy link

EB not only searches for *-GCC-* but also for *-GCCcore-* if no file was found using the former

EB not only searches for `*-GCC-*`  but also for `*-GCCcore-*` if no file was found using the former
@JDiprose
Copy link

JDiprose commented Jul 3, 2024

A link to https://docs.easybuild.io/common-toolchains/#toolchains_diagram might also help here, as a clear picture of the toolchain layers. Perhaps:
EB also searches for modules of the same name and version that use lower-level toolchain components from the same chain, eg -GCCcore- for an original -GCC-. See the toolchain documentation | https://docs.easybuild.io/common-toolchains/#toolchains_diagram for a fuller explanation.
I'll see if I can suggest that in the file.

@JDiprose
Copy link

JDiprose commented Jul 3, 2024

Ah, you've already got it! This would have helped me.

@@ -528,6 +528,15 @@ For example, when looking for an easyconfig for `OpenMPI` version `4.1.4` and ve
* `o/OpenMPI/OpenMPI-4.1.4-GCC-11.3.0-test.eb`
* `OpenMPI-4.1.4-GCC-11.3.0-test.eb`

Copy link

@JDiprose JDiprose Jul 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that it is not an indefinite-depth search, so if you have cloned the easybuilders/easybuild-easyconfigs git repo then you need 'easybuild-easyconfigs/easybuild/easyconfigs' on your robot-path.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just added c7567cd
Is that comprehensible enough?

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.

None yet

2 participants