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.

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

unported path in postnek's makefile #16

Closed
maxhutch opened this issue Jun 29, 2016 · 3 comments
Closed

unported path in postnek's makefile #16

maxhutch opened this issue Jun 29, 2016 · 3 comments

Comments

@maxhutch
Copy link
Contributor

This line was only half ported: the blas dependency still points to nek/blas instead of core/blas

@RonRahaman
Copy link
Contributor

RonRahaman commented Jul 1, 2016

@maxhutch, is nek5000/3rd_party/ an artifact? Should it be deleted?

@maxhutch
Copy link
Contributor Author

maxhutch commented Jul 1, 2016

No. That's a landing point for 3rd_party add-ons, e.g. KTH's toolbox. It
should stay empty.

On Fri, Jul 1, 2016 at 11:18 AM Ron Rahaman notifications@github.com
wrote:

@maxhutch https://github.com/maxhutch, nek5000/3rd_party/ an artifact?
Should it be deleted?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#16 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/ABd7pGh1p7pczw-Ul1KfO_2VHOJlcPkjks5qRS-pgaJpZM4JBS-X
.

@maxhutch maxhutch added this to the First stable git release milestone Jul 12, 2016
@maxhutch
Copy link
Contributor Author

Fixed in #22

RonRahaman pushed a commit to RonRahaman/Nek5000 that referenced this issue Nov 21, 2016
I believe I have satisfied the requirements laid forth by the nek community to bring CMT-nek into v16.0.0.0. Compiling nek5000 now proceeds without any cmt-nek code at all; it all lives in core/cmt. IFCMT has been replaced by CMTNEK, a compiler directive that appears in three places in two source files. travis and NekTests.py reflect this accurately. Let me know if I need to do anything else. Thank you for your patience.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants