search

OPM / opm-material

Provides thermodynamic relations, capillary pressure curves, etc.
Other
15 stars 51 forks source link

Raising a fatal error by default in cmake. #548

Closed kjetilly closed 1 year ago

kjetilly commented 1 year ago

We've had some unfortunate incidences where people have used automated build scripts to build OPM, but have not been notified of the deprecation of this module. As it stands, opm-material will fail to compile due to some dependency not being met.

This PR addes a fatal error at the top of CMakeLists.txt such that running cmake <path to opm-material> results in 

CMake Error at CMakeLists.txt:26 (message):
  THIS MODULE IS DEPRECATED.  The material module have been integrated into
  opm-common.

If one really wants to build it, one can specify -DBUILD_DEPRECATD_OPM_MATERIAL=ON and have it built (provided one can find matching versions of the other repositories).