-
Notifications
You must be signed in to change notification settings - Fork 312
resolve templates in CmdCp easyblock
#3918
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
resolve templates in CmdCp easyblock
#3918
Conversation
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 8 out of 8 (1 easyconfigs in total) |
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 12 out of 13 (10 easyconfigs in total) |
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 14 out of 15 (11 easyconfigs in total) |
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 8 out of 8 (7 easyconfigs in total) |
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 15 out of 15 (15 easyconfigs in total) |
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 8 out of 8 (7 easyconfigs in total) |
boegel
left a comment
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.
lgtm
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 11 out of 11 (9 easyconfigs in total) |
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 0 out of 3 (3 easyconfigs in total) |
|
Test report by @branfosj Overview of tested easyconfigs (in order)
Build succeeded for 6 out of 7 (7 easyconfigs in total) |
|
Failures from this PR:
Failures not related to this PR:
|
| for pattern, regex_cmd in self.cfg.get_ref('cmds_map'): | ||
| pattern = pattern % self.cfg.template_values | ||
| regex_cmd = regex_cmd % self.cfg.template_values |
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.
Isn't that equivalent? The getter recursively resolves the templates so how is this different than before?
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.
Quick print shows that those are equivalent, so reverting this in #3921
Can you check that PR?
CmdCpCmdCp easyblock
(created using
eb --new-pr)Completes the fix for easybuilders/easybuild-framework#4990
The switch to
self.cfg.get_ref('cmds_map')fixesThe resolving of the
patterntemplate variables fixes