Build failure caused by "Add eclipse format output to SteadyStateUpscalerImplicit" commit about opm-upscaling HOT 5 CLOSED

The intent is that Opm::writeECLData() should be available for linking even if HAVE_ERT is defined (throwing an exception if actually called). This is done to reduce the configuration complexity of opm-upscaling. However, there is a conditional in the build system of opm-core that I forgot to remove. I'll fix this shortly.

from opm-upscaling.

I have now submitted a pulll request to opm-core to fix this. Please test it by checking out the branch no-ert-conditional in my opm-core fork.

from opm-upscaling.

I get a compile error for opm-core with the no-ert-conditional branch:

opm/core/utility/writeECLData.cpp:158:29: error: 'SimulatorTimer' does not name a type
opm/core/utility/writeECLData.cpp:158:43: error: ISO C++ forbids declaration of 'parameter' with no type [-fpermissive]

Should there perhaps be a

#include <opm/core/simulator/SimulatorTimer.hpp>

after line 152 of opm/core/utility/writeECLData.cpp?

from opm-upscaling.

@Ivarun: this should fix your latest issue. Assigned to you for closing if no more problems (feel free to reassign if it still fails).

from opm-upscaling.

Compiles fine now, thank you.

from opm-upscaling.