search

MesserLab / SLiM

SLiM is a genetically explicit forward simulation software package for population genetics and evolutionary biology. It is highly flexible, with a built-in scripting language, and has a cross-platform graphical modeling environment called SLiMgui.
https://messerlab.org/slim/
GNU General Public License v3.0
160 stars 30 forks source link

provide `make test` functionality to run tests after building `slim` and `eidos` #438

Closed bhaller closed 1 month ago

bhaller commented 5 months ago

@bryce-carson wrote in #437 (the *** buffer overflow detected *** fallout from the -D_FORTIFY_SOURCE=3 change in some Linux systems):

@bhaller, if you can write a new section in the Makefile to run the tests, so that make test can be called, that'd be useful for the packaging process to detect issues like these in the future (it's not needed for the patched RPM; that's already live). I tried to locate the binaries and call the tests myself, but I couldn't find them where I expected them to be. It's easier to hand the work off to CMake since it magically knows where the binaries are.

This issue represents that request. Presumably a fairly simple CMake change, but I'm not very fluent with CMake so who knows. :-> If somebody else reading this knows what to do (tagging @grahamgower as well), feel free to submit a PR. :->

bhaller commented 1 month ago

OK, with #449 now merged, I think this issue is now fixable...? Let me know if there's any obstacle remaining. Thanks!

bhaller commented 1 month ago

I guess this issue is completed and should be closed. @bryce-carson if you do intended to use this make test functionality to make the RPM builds more robust, as discussed elsewhere, please feel free to make a new issue, or just have it on your own personal to-do list, as you wish. :-> Thanks all!