Skip to content
New issue

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

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

Already on GitHub? # to your account

enable PerlModules that provide multiple modules #6

Conversation

lexming
Copy link

@lexming lexming commented Apr 16, 2021

Suggestion for your PR easybuilders#2386

@Flamefire
Copy link
Owner

In the PR you said e.g. IO:* is part of perl base. How would those be handled with this change?

I wouldn't mind combining both, i.e. allowing string-only items and multiple module names.

However I remember ECs being type-checked. Would this "abuse" of modulename throw this off? @boegel?

@boegel
Copy link

boegel commented Apr 16, 2021

It would be better to implement support for modulename being a list of values in EasyBuild framework.

This is hacky at best, since it also does the rest of the sanity check, not just the "import" check...

@lexming
Copy link
Author

lexming commented Apr 20, 2021

I targeted the easyblock of PerlModule because this might only be necessary with Perl. I'm not aware of any group of packages in R or Python being distributed in a single source tarball. So the goal is just to emulate single-module virtual extensions and use existing facilities in framework.

If you prefer to have general support for lists in modulename, them that's another story. I'll take a look in framework to see how much work is needed to get that done.

@lexming
Copy link
Author

lexming commented Jan 17, 2023

Abandoned. Code will stay available at https://github.com/lexming/easybuild-easyblocks/tree/multi_perlmod

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants