dolfin team mailing list archive

[Johan Jansson <johanjan@xxxxxxxxxxxxxxxx>]

On Tue, Aug 29, 2006 at 08:44:26AM +0200, Garth N. Wells wrote:
> > changeset:   2188:2b5ec95a0799d4c4952fa7cdcfec918259c28b7d
> > user:        "Johan Jansson <johanjan@xxxxxxxxxxxxxxxx>"
> > date:        Tue Aug 29 07:42:23 2006 +0200
> > files:       configure.ac src/config/Makefile.am
> > description:
> > Build:
> > 
> > Define EXTERNAL_CFLAGS as includes for external libraries. Add
> > EXTERNAL_CFLAGS to "dolfin-config --cflags" (necessary for PETSc for
> > instance).
> > 
> 
> 
> Why and where is this necessary? This causes problems when PETSc and
> DOLFIN use a different version of the same library (as is the case with
> UMFPACK).
> 
> Garth

Ok, I didn't know that. Maybe I've misunderstood something, but the
demos didn't compile for me. For example, in the Poisson demo
src/demo/pde/poisson/Makefile:

CFLAGS  = `dolfin-config --cflags`
...
$(DEST): $(OBJECTS)
         $(LINK) -o $@ $(OBJECTS) $(CFLAGS) $(LIBS)

If CFLAGS doesn't include any include paths (to PETSc for instance),
then it won't be able to find the headers for library dependencies.

Did this work for you before I changed it? It seems it was changed in
this changeset:

http://www.fenics.org/hg/dolfin?cs=866f7f5da06f

I assumed this was an oversight since uBlas is standard now instead of
PETSc.

  Johan