Skip to content

/ casadi

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

Jump to bottom

Install SWIG interfaces to CMAKE_INSTALL_PREFIX/language by default #1991

Closed
jaeandersson opened this issue Apr 25, 2017 · 1 comment
Closed

Install SWIG interfaces to CMAKE_INSTALL_PREFIX/language by default #1991

jaeandersson opened this issue Apr 25, 2017 · 1 comment
Assignees
@jaeandersson
Labels
enhancement
Milestone
Version 3.2

Comments

@jaeandersson
Copy link
Member

@jaeandersson jaeandersson commented Apr 25, 2017

The default installation directory for e.g. the Python module should be CMAKE_INSTALL_PREFIX/python, for Octave CMAKE_INSTALL_PREFIX/octave etc. This avoids accidental installation into system paths and makes it easier to switch between multiple versions of CasADi.
Cf. #1982
@jaeandersson jaeandersson self-assigned this Apr 25, 2017
@jaeandersson
Copy link
Member Author

@jaeandersson jaeandersson commented Apr 25, 2017

Both MATLAB and Octave will be installed into CMAKE_INSTALL_PREFIX/matlab, as they should not conflict and have a lot of common code.
jaeandersson added a commit that referenced this issue Apr 25, 2017
@jaeandersson
Issue #1991 Removed dead code in swig/python/CMakeLists.txt
a452b97
jaeandersson added a commit that referenced this issue Apr 25, 2017
@jaeandersson
Issue #1991 #704 Removed legacy workaround
e1c6acf
jaeandersson added a commit that referenced this issue Apr 25, 2017
@jaeandersson
Issue #1991 Refactored installation directories
0aeb4ec
jaeandersson added a commit that referenced this issue Apr 25, 2017
@jaeandersson
Issue #1991 Refactored MATLAB_PREFIX (for both Octave and MATLAB)
ff9d347
@jaeandersson jaeandersson added this to the Version 3.2 milestone May 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jaeandersson jaeandersson

Labels
enhancement
Projects
None yet
Milestone
Version 3.2
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@jaeandersson