Hide Forgot
Spec URL: https://ankursinha.fedorapeople.org/neuron/neuron.spec SRPM URL: https://ankursinha.fedorapeople.org/neuron/neuron-7.5-1.20181214git5687519.fc29.src.rpm Description: NEURON is a simulation environment for modeling individual neurons and networks of neurons. It provides tools for conveniently building, managing, and using models in a way that is numerically sound and computationally efficient. It is particularly well-suited to problems that are closely linked to experimental data, especially those that involve cells with complex anatomical and biophysical properties. This package is currently built without GUI (iv) support. It does not include MPI support. Fedora Account System Username: ankursinha
*** Bug 1249094 has been marked as a duplicate of this bug. ***
# Let us tell the system where to find the sundials libraries. It is hard coded. sed -i 's|SUNDIALS_LIBDIRNAMES="${prefix}/lib"|SUNDIALS_LIBDIRNAMES="$MPI_LIB"|' configure.ac You're doing the above but not module-loading any MPI implementation configuration, nor is any MPI library added to build dependencies. Could you sort the BuildRequires and put each one on a separate line? I would use 'rm' to delete Random123 files instead of doing that with a patch. It'd make the patch much smaller.
Hi Dominik, Thanks for taking up the review. I've updated the spec/srpm: * Sun Jan 06 2019 Ankur Sinha <ankursinha AT fedoraproject DOT org> - 7.5-2.20181214git5687519 - Put each BR on different line - Remove unneeded comment - Re-do random123 patch to only modify autotools files - Remove random123 in prep https://ankursinha.fedorapeople.org/neuron/neuron.spec https://ankursinha.fedorapeople.org/neuron/neuron-7.5-2.20181214git5687519.fc29.src.rpm This is a serial (non MPI) build of NEURON. The MPI builds, and python bindings require NEURON to be installed already. Upstream builds them as post-installation hooks. So, I'll package them up as different packages that will require this one as a BR. I've added this as a comment at the start of the spec. Cheers, Ankur