Skip to content

#302 wrong usage of config.h

Metadata

Property Value
Reported by Christian Engwer (christi@conan.iwr.uni-heidelberg.de)
Reported at Jul 17, 2007 13:27
Type Bug Report
Version Git (pre2.4) [autotools]
Operating System Unspecified / All
Closed by Oliver Sander (oliver.sander@tu-dresden.de)
Closed at Jul 24, 2007 13:03
Closed in version 1.0
Resolution Fixed
Comment Fixed in dune-disc revision 438.

Description

  • header must not include config.h, this includes cc-files, if they are included by header files.

  • cc-files should include config.h, if they are compiled into object files.

  • config.h should be incuded via #include <config.h>

I think these bugs are easy to fix and are quite severe bugs, therefor I marked this bug "1.0". These bugs will render Dune without config.h unusable.

dune-grid/trunk and dune-disc/releases/1.0: ./grid/io/file/amiramesh/amuggridreader.cc:#include <dune/config.h>

dune-disc/trunk and dune-disc/releases/1.0: ./disc/stokes/dgstokes.cc:# include "config.h" // autoconf defines, needed by the dune headers ./disc/stokes/testfunctions.cc:# include "config.h" // autoconf defines, needed by the dune headers ./disc/stokes/rhs.cc:# include "config.h" // autoconf defines, needed by the dune headers ./disc/stokes/boundaryconditions.cc:# include "config.h" // autoconf defines, needed by the dune headers ./disc/shapefunctions/lagrangeshapefunctions.cc:#include "config.h"