-
-
Notifications
You must be signed in to change notification settings - Fork 544
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
Use sdh_cmake for packages that use cmake #27486
Milestone
Comments
comment:1
neither qhull nor cryptominisat try to install anything in lib64 or so, thus it's really not that urgent to fix. |
Branch: u/chapoton/27486 |
Commit: |
New commits:
|
comment:3
please have a look |
Author: Frédéric Chapoton |
comment:4
looks good to me |
Reviewer: Dima Pasechnik |
Changed branch from u/chapoton/27486 to |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
#27484 adds an
sdh_cmake
helper function for use by our handful of (all currently optional/experimental) packages. We should try updating those packages to use the new helper:Of these, only scipoptsuite is somewhat unusual in that it does a sort of VPATH build in that it cds into a
build/
directory and then runscmake ..
. This is incompatible with the current implementation ofsdh_cmake
. So for that package either we don't usesdh_cmake
, or forego the use of thebuild/
directory, which may not be necessary anyways.CC: @embray @jdemeyer
Component: packages: optional
Author: Frédéric Chapoton
Branch/Commit:
1df46ef
Reviewer: Dima Pasechnik
Issue created by migration from https://trac.sagemath.org/ticket/27486
The text was updated successfully, but these errors were encountered: