Skip to content
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.

Sign up for GitHub

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

Jump to bottom

lmod (7.6.1) #495

Closed
koomie opened this issue Jul 26, 2017 · 4 comments
Closed

lmod (7.6.1) #495

koomie opened this issue Jul 26, 2017 · 4 comments
Assignees
@koomie
Labels
built component t:admin
Milestone
1.3.2

Comments

@koomie
Copy link
Contributor

koomie commented Jul 26, 2017 •
edited
Loading

https://github.com/TACC/Lmod/releases

This should allow access to new functionality that can be used to define dependencies amongst modules.
@koomie koomie added this to the 1.3.2 milestone Jul 26, 2017
@koomie koomie self-assigned this Jul 26, 2017
koomie added a commit that referenced this issue Jul 26, 2017
@koomie
bumping lmod version to v7.5.17 (#495)
02a3f82
@fgeorgatos
Copy link

fgeorgatos commented Jul 27, 2017 via email

koomie added a commit that referenced this issue Jul 27, 2017
@koomie
Adding patch to enable "depends-on" keyword support for tcl modulefil…
ed8da80 
…e parsing

via Lmod (#495)
koomie added a commit that referenced this issue Jul 27, 2017
@koomie
patches have a better chance of doing something useful if you enable …
a429ade 
…them (#495)
@koomie
Copy link
Contributor Author

koomie commented Jul 27, 2017

Looks like the depends_on() functionality was not extended to tcl files. Issued a PR upstream to enable so we can use in ohpc (TACC/Lmod#303).

@koomie
Copy link
Contributor Author

koomie commented Aug 2, 2017

v7.6.1 is out that should have the tcl fix to support depends_on.

@koomie koomie changed the title lmod (7.5.17) lmod (7.6.1) Aug 2, 2017
koomie added a commit that referenced this issue Aug 2, 2017
@koomie
bumpt Lmod to version v7.6.1 (#495)
dfb027c
koomie added a commit that referenced this issue Aug 2, 2017
@koomie
depends_on patch was upstreamed (#495)
8916ab1
@koomie koomie added the built label Aug 2, 2017
crbaird added a commit that referenced this issue Aug 7, 2017
@crbaird
update rpm requirements for depends-on (#495)
9b47c90
crbaird added a commit that referenced this issue Aug 7, 2017
@crbaird
update rpm requirements for depends-on (#495)
4dd1052
@koomie
Copy link
Contributor Author

koomie commented Aug 10, 2017

CI tests passing.

@koomie koomie closed this as completed Aug 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@koomie koomie

Labels
built component t:admin
Projects
None yet
Milestone
1.3.2
Development

No branches or pull requests
2 participants
@koomie @fgeorgatos