-
Notifications
You must be signed in to change notification settings - Fork 24
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
patch CMakeLists #73
patch CMakeLists #73
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.
What's the symptom of the current problematic configuration? Is this the most idiomatic way to address the symptom?
use ${CMAKE_INSTALL_LIBDIR} variable Co-authored-by: Dan N <dan.nealschneider@schrodinger.com>
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.
Thanks!
Hm, I think we need to update some of the CI build instructions |
for reference, the Fedora package is visible at https://src.fedoraproject.org/rpms/maeparser |
This mirrors schrodinger/maeparser#73 in maeparser
Thanks for this, @mefuller By the way - who writes the package documentation? Is there some metadata that we should add here that would make it easier for package managers like yourself to automatically generate the documentation? It looks like it's independently written, but based on the README. |
Independently generated - I copied it in. |
This mirrors schrodinger/maeparser#73 in maeparser
Modify CMakeLists.txt to place files in correct lib dir (e.g. on lib64 systems)