-
Notifications
You must be signed in to change notification settings - Fork 202
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
avoid needless use of deepcopy, speed up support for templating in easyconfigs #1897
Conversation
… instance and avoiding to reparse easyconfig for toolchain
…ain' easyconfig parameter
…ging provided easyconfigs
@@ -323,7 +325,7 @@ def __init__(self, path, extra_options=None, build_specs=None, validate=True, hi | |||
|
|||
# list of *all* dependencies, including hidden/build deps & toolchain, but excluding filtered deps | |||
self.all_dependencies = copy.deepcopy(self.dependencies()) | |||
if self.toolchain.name != DUMMY_TOOLCHAIN_NAME: | |||
if self['toolchain']['name'] != DUMMY_TOOLCHAIN_NAME: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this a pure style change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, it's avoiding that a Toolchain
instance is created already at this point, which is not needed at all for dummy
toolchains here...
…compiled regex's anyway
Thanks for the review @wpoely86! |
These changes are motivated by profiing the
easyconfigs
test suite, which was starting to take prohibitively long (> 10m).Together with easybuilders/easybuild-easyconfigs#3520, this speeds up the
easyconfigs
test suite by a factor 3x...