Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

Build issues caused by configuration mismatches or missing header #47

Closed
atgeirr opened this issue Feb 12, 2013 · 3 comments
Closed

Build issues caused by configuration mismatches or missing header #47

atgeirr opened this issue Feb 12, 2013 · 3 comments

Comments

@atgeirr
Copy link
Contributor

atgeirr commented Feb 12, 2013

In a different library (opm-core) I get the following error message:

In file included from /Users/atgeirr/opm/opm-core/examples/import_rewrite.cpp:11:0:
/usr/local/include/ert/util/util.h:435:23: fatal error: util_fork.h: No such file or directory

This line is compiled because my (opm-core) config.h contains the line

#define HAVE_FORK 1

However, util_fork.h does not get installed by ert. The ert_build_config.h file contains

#define HAVE_FORK

without the 1, I do not know if this is significant.

In general, I guess opm-core must make a configuration header that is a perfect superset of ert's header, or I can get into trouble. The best would be to have no configuration dependencies in any installed ert header, but that may not be feasible.

@joakim-hove
Copy link
Contributor

OK;

see merged PR: #49. The PR removes the -fno-leading-underscore which was some ancient desperate windows attempt.

The content of (missing) header file util_fork.h is inlined in the main header file util.h, and things 'should work' now - however you are addressing an important issue with the build config header (which is not used at all for the moment - I think ....). My experience from ert is of course mainly to build and use ert itself, not building other codes which depend on ert; I therefor fully understand that ert does not behave optimally for other libraries. I will

@joakim-hove
Copy link
Contributor

ehh - a bit premature send here:

My point is that I am aware that way is build config is exported from ert to e.g. opm-core is not 100%, and that I will be more than happy to fix it; suggestions accepted.

@atgeirr
Copy link
Contributor Author

atgeirr commented Feb 13, 2013

This issue is now resolved, thanks!

@atgeirr atgeirr closed this as completed Feb 13, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants