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

remove dune-control support #36

Closed
akva2 opened this issue Sep 8, 2015 · 7 comments
Closed

remove dune-control support #36

akva2 opened this issue Sep 8, 2015 · 7 comments

Comments

@akva2
Copy link
Member

akva2 commented Sep 8, 2015

dune-control support is a useless hack and maintaining support is additional load on the build system.

let's remove this.

@dr-robertk
Copy link
Member

@akva2: Ok. But only if the sibling stuff goes too. I'm using dune-control and to that date it works fine and is the only way that works to build the OPM modules. I'm pretty sure that when I follow the instructions on the OPM web site, that I won't be able to build OPM. Until then, let's keep it.

@bska
Copy link
Member

bska commented Sep 8, 2015

I'm using dune-control and to that date it works fine and is the only way that works to build the OPM modules.

That statement is not true--at least not literally. I'm building all of OPM in many different configurations and I don't use dunecontrol.

@atgeirr
Copy link
Member

atgeirr commented Sep 8, 2015

But only if the sibling stuff goes too.

Please don't take hostages...

Dune-control support and sibling builds will remain for the time being since we have not yet reached a consensus about a single proper way to build.

I'm pretty sure that when I follow the instructions on the OPM web site, that I won't be able to build OPM

That has (unfortunately) been true since we introduced opm-cmake. However they have been tested (by people here coming into the project) successfully a few times. I will update the instructions when opm-common is ready.

@andlaus
Copy link
Contributor

andlaus commented Sep 8, 2015

I'm using dunecontrol all the time. I would not mind if support for it was removed though, if I had an adequate alternative to rebuild everything with different options in one go.

@dr-robertk
Copy link
Member

I investigated this topic a bit and it turns out that everything works fine when DuneCompat.cmake is removed (DUNE 2.4 with cmake). The configure script in cmake/Scripts is not needed anymore, because dunecontrol calls cmake directly and also not need is the DuneCompat2.cmake. There is a minor issue in dunecontrol that is being fixed as we "speak".

The only problem I see, is, that this would require to use the latest release version, DUNE version 2.4. But it might take the OPM community another decade or so to upgrade to 2.4.....

@atgeirr
Copy link
Member

atgeirr commented Sep 16, 2015

The only problem I see, is, that this would require to use the latest release version, DUNE version 2.4. But it might take the OPM community another decade or so to upgrade to 2.4.....

I don't think so: we can probably leapfrog to 2.4 at some point instead of going through 2.3. @alfbr?

@atgeirr
Copy link
Member

atgeirr commented Oct 9, 2015

Dune-control support and sibling builds will remain for the time being since we have not yet reached a consensus about a single proper way to build.

This I said a while ago, and I will therefore close this issue. However:

I investigated this topic a bit and it turns out that everything works fine when DuneCompat.cmake is removed (DUNE 2.4 with cmake). The configure script in cmake/Scripts is not needed anymore, because dunecontrol calls cmake directly and also not need is the DuneCompat2.cmake. There is a minor issue in dunecontrol that is being fixed as we "speak".

Therefore: when we are on Dune 2.4 we should revisit and simplify a bit (removing the configure script etc.), but for now this is a non-issue.

@atgeirr atgeirr closed this as completed Oct 9, 2015
akva2 pushed a commit to akva2/opm-common that referenced this issue Jun 10, 2022
EclDefaultMaterial: use inconsistent saturations for the hysteresis update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants