Skip to content
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

CMake: Unset CMAKE_INSTALL_PREFIX #113

Closed
wants to merge 1 commit into from

Conversation

jschueller
Copy link
Contributor

Closes #12

@stefansli
Copy link

@PeterMeisrimelModelon @modelonrobinandersson @MikaelJohnssonModelon please asap. It's really cumbersome to include fmilibrary in any package management system. If the community tries to fix the issues and creates pull requests it's very sad to see those pull requests gett no attendion for months.

@modelonrobinandersson
Copy link
Collaborator

Hi @stefansli and @jschueller.
I just opened #126 to include 113, 114 and some other updates which simplifies this a bit. As soon as I get all the tests to pass on Linux, Windows (MinGW and MSVC (MD/MT)) I will merge it.

I've also seen the other PRs but I will have to take them later as they cause some issues with older compilers on Windows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Overriding CMAKE_INSTALL_PREFIX is bad practice
3 participants