Skip to content

Conversation

@bartoldeman
Copy link
Contributor

This reintroduces commit 81ec7cc which was partially undone by
commit df69253, including the testcase.
This commit reintroduces the old testcase after the new one with
a different name (test_make_module_dep_of_dep_hmns).

This reintroduces commit 81ec7cc which was partially undone by
commit df69253, including the testcase.
This commit reintroduces the old testcase after the new one with
a different name (test_make_module_dep_of_dep_hmns).
@bartoldeman
Copy link
Contributor Author

Travis failed because of an internal server error downloading Lmod 6.6.3 from github.

@boegel
Copy link
Member

boegel commented Mar 9, 2017

@bartoldeman Yes, I'm seeing that download problem for Lmod 6.6.3 pop up in other places too on Travis (which is weird, since it works fine for other Lmod versions...).

Clearly unrelated to this PR. :)

@boegel boegel added this to the 3.1.2 milestone Mar 9, 2017
# load modules that open up the module tree before checking deps of deps (in reverse order)
self.modules_tool.load(excluded_deps[::-1])

for dep in excluded_deps:
Copy link
Member

@boegel boegel Mar 9, 2017

Choose a reason for hiding this comment

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

@bartoldeman one tiny suggestion: I would rename dep to excluded_dep here, to avoid confusion...

@boegel
Copy link
Member

boegel commented Mar 9, 2017

@akesandgren Since you also reported seeing load GCCcore statements being included with EasyBuild v3.1.1, can you verify that this fixes the issue on your side as well?

@akesandgren
Copy link
Contributor

Yep, new patch works, GCCcore no longer present in the module generated from my ScaLAPACK-2.0.2-intel2017a.eb

Nicely done.

@boegel
Copy link
Member

boegel commented Mar 10, 2017

Thanks for the fix @bartoldeman, and @akesandgren for the confirmation!

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.

3 participants