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

Allow DUNE modules to locate the dune.module files during CMake run. #302

Merged
merged 1 commit into from
Jun 18, 2018

Conversation

blattms
Copy link
Member

@blattms blattms commented Dec 12, 2017

For this we need to set _PREFIX to some ominous value (at least for installed versions). I have no idea what I was thinking when implementing and naming it like this in DUNE. The name is actually correct, I just did not see it. _PREFIX either set to where the module is installed to or the source directory for a build with uninstalled modules.

This is an untested shot to resolve the problem described in #300.

@blattms blattms force-pushed the allow-dune-to-find-dune.module branch 2 times, most recently from 322cfbd to 6d03c27 Compare December 12, 2017 15:56
@blattms
Copy link
Member Author

blattms commented Dec 12, 2017

Tested with a play dune module (created by duneproject) depending on opm-grid with installed and uninstalled opm-grid.

For this we need to set <dune-module>_PREFIX either to where the module
is installed to or the source directory for a build with uninstalled
modules.
@blattms blattms force-pushed the allow-dune-to-find-dune.module branch from 6d03c27 to c6536c0 Compare June 18, 2018 10:48
@blattms
Copy link
Member Author

blattms commented Jun 18, 2018

Rebased and testes with patched dumux version. See OPM/opm-grid#327

@blattms
Copy link
Member Author

blattms commented Jun 18, 2018

jenkins build this please

@akva2 akva2 merged commit 3085243 into OPM:master Jun 18, 2018
@blattms blattms deleted the allow-dune-to-find-dune.module branch June 18, 2018 12:17
akva2 pushed a commit to akva2/opm-common that referenced this pull request Jun 10, 2022
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.

2 participants