feat(nest): update nest extension installation dir#782
feat(nest): update nest extension installation dir#782sanjayankur31 wants to merge 1 commit intoNeuralEnsemble:masterfrom
Conversation
The `/nest` bit seems to be included in `NEST_LIBDIR` now, so does not need to be explicitly added: https://github.com/nest/nest-extension-module/blob/master/CMakeLists.txt#L178 I expect `nest-config --libdir` provides the complete path already.
|
@sanjayankur31 for me Do you see something different? |
|
I haven't tested just in a virtualenv. This is for the Fedora build where we explicitly set the CMAKE variables etc.. We do set I do see that in the latest code for the nest-extension-module, they don't use the but maybe this patch isn't really required upstream (and we just carry it downstream so that pynn matches what we do with nest)? |
The
/nestbit seems to be included inNEST_LIBDIRnow, so does not need to be explicitly added:https://github.com/nest/nest-extension-module/blob/master/CMakeLists.txt#L178
I expect
nest-config --libdirprovides the complete path already.