-
Notifications
You must be signed in to change notification settings - Fork 407
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
Install hpcbind script: issue #3304 #3402
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
@crtrott , do we want to maintain these scripts, and promise their effectiveness? If yes, I think this is good, but that's still the question we haven't answered |
who is using hpcbind and who is volunteering to maintain it? This was Dan's baby. @nmhamster Si do you think this is a good idea or bad idea (hpcbind)? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just putting a block on this, I want to hear from @nmhamster how he feels about the script.
We had a user in Slack request this a month ago or so, I filed the tracking issue and helpfully didn't name them (oops). I'll see if I can find them in our history |
5132844
to
46830db
Compare
Talked with nmhamster and he agrees its good to install this. Just need to resolve the merge conflict. |
Co-Authored-By: Jeremiah Wilke <jjwilke@sandia.gov>
8959420
to
92f8541
Compare
Take advantage of the CMake `install` command support Co-Authored-By: Jeremiah Wilke <jjwilke@sandia.gov>
92f8541
to
e158706
Compare
Failure is unrelated |
No description provided.