Skip to content

Commit

Permalink
Bundle: Address review comments
Browse files Browse the repository at this point in the history
Signed-off-by: Jan André Reuter <j.reuter@fz-juelich.de>
  • Loading branch information
Thyre committed Dec 4, 2024
1 parent 43ec355 commit bce45f5
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions easybuild/easyblocks/generic/bundle.py
Original file line number Diff line number Diff line change
Expand Up @@ -322,15 +322,15 @@ def install_step(self):

def make_module_req_guess(self):
"""
Set module requirements from all comppnents, e.g. $PATH, etc.
Set module requirements from all components, e.g. $PATH, etc.
During the install step, we only set these requirements temporarily.
Later on when building the module, those paths are not considered.
Therefore, iterate through all the components again and gather
the requirements.
Do not remove duplicates or check for existance of folders,
Do not remove duplicates or check for existence of folders,
as this is done in the generic EasyBlock while creating
the modulefile already.
the module file already.
"""
# Start with the paths from the generic EasyBlock.
# If not added here, they might be missing entirely and fail sanity checks.
Expand All @@ -342,10 +342,10 @@ def make_module_req_guess(self):

try:
for key, value in sorted(reqs.items()):
if isinstance(reqs, string_type):
if isinstance(value, string_type):
value = [value]
final_reqs.setdefault(key, [])
final_reqs[key] += value
final_reqs[key].append(value)
except AttributeError:
raise EasyBuildError("Cannot process module requirements of bundle component %s v%s",
cfg['name'], cfg['version'])
Expand Down

0 comments on commit bce45f5

Please sign in to comment.