Skip to content

cp2k : added dependencies#1724

Merged
tgamblin merged 2 commits intospack:developfrom
epfl-scitas:packages/cp2k_dependencies
Sep 21, 2016
Merged

cp2k : added dependencies#1724
tgamblin merged 2 commits intospack:developfrom
epfl-scitas:packages/cp2k_dependencies

Conversation

@alalazo
Copy link
Copy Markdown
Member

@alalazo alalazo commented Sep 4, 2016

Modifications
  • added wannier90 and elpa as dependencies of cp2k

* wannier90 : added package

* cp2k : added dependency on wannier90
@alalazo
Copy link
Copy Markdown
Member Author

alalazo commented Sep 4, 2016

@tgamblin If you could merge #1264 from @adamjstewart I also integrated that into cp2k

'-L' + spec['scalapack'].prefix.lib
])
libs.extend([
join_path(spec['elpa'].prefix.lib, 'libelpa.so'),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

.so won't work on macOS.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Definitely a place for find_libraries from #1682

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done along with other two dso_suffix that I overlooked... thanks

Conflicts:
	var/spack/repos/builtin/packages/elpa/package.py
@alalazo alalazo force-pushed the packages/cp2k_dependencies branch from 6f22bf2 to 4275340 Compare September 5, 2016 08:08
@tgamblin tgamblin merged commit 3254d2f into spack:develop Sep 21, 2016
@alalazo alalazo deleted the packages/cp2k_dependencies branch September 21, 2016 07:56
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