Simplify and optimise build process#272
Open
davexparker wants to merge 16 commits intoprismmodelchecker:masterfrom
Open
Simplify and optimise build process#272davexparker wants to merge 16 commits intoprismmodelchecker:masterfrom
davexparker wants to merge 16 commits intoprismmodelchecker:masterfrom
Conversation
And remove Makefiles that are unnecessary as a result.
prism/Makefile is now redundant and removed.
Speeds up "make" a lot (~80% reduction).
Set soname/install_name for libprism and Mac liblpsolve55/liblpsolve55j, where it was previously missing. For shared libraries with dependencies (libprism, liblpsolve55j), we also set rpath so that the linker looks in the same directory for dependencies. This means that LD_LIBRARY_PATH and DYLD_LIBRARY_PATH are removed from the bin/prism scripts for Linux/Max, just using -Djava.library.path instead. This fixes a bunch of issues with Java on Mac (e.g. using jenv) and makes things generally cleaner and more portable. Also no need to set up the (DY)LD_LIBRARY_PATH variables for running through IDEs. Windows/Cygwin is unaffected and continues to need to set/use PATH.
This was referenced Feb 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
src/libprism.{so,dylib}makeprocess, minimising variable evaluationsoname/install_nameandrpathon shared libraries(no need for (
DY)LD_LIBRARY_PATHon Mac/Linux now)