-
Notifications
You must be signed in to change notification settings - Fork 357
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
Make helper libraries static and don't install the archive files #2655
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 fine to me!
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.
After checking the changes in #2250 again, I am pretty sure, we don't need the VERSION ${NEST_VERSION}
lines anymore now that the libraries are static. @sanjayankur31, feel free to contradict me -- either here or in a follow-up issue or pull request.
Sorry---been away on leave. Yes, versioning is only required for the ones that are being included as shared objects. Bits that are internal static libs don't need it. |
This fixes #1182 by making
libmodels
,libnestutil
andlibnestkernel
static libraries that are directly included wherever they are needed. As these are only internal helper libraries, there is no need to install them.The remaining libraries
libnest
,libsli
andlibsli_readline
will be taken care of by the PyNEST NG project.@sanjayankur31: I assume that with these changes, also the
VERSION ${NEST_VERSION}
line could go away, can't it?