On 15 May 2013 17:12, Satish Balay <balay at mcs.anl.gov> wrote: > On Wed, 15 May 2013, Lisandro Dalcin wrote: > >> On 15 May 2013 16:54, Satish Balay <balay at mcs.anl.gov> wrote: >> > On Wed, 15 May 2013, Matteo Parsani wrote: >> > >> >> Hello Satish, >> >> Lisandro has found that in the make file we have >> >> >> >> include ././${PETSC_ARCH}/conf/petscvariables >> > >> > we try to support too many ways of installing petsc [with prefix/ >> > without prefix, with a defaut PETSC_ARCH, without PETSC_DIR set etc..] >> > And we need this line for users who forget to set PETSC_ARCH in a >> > non-prefix build. [to pick up a defaut PETSC_ARCH] >> > >> >> >> Satish, I think all what is needed is to fix a little the test_build >> target. Basically, test if the file under PETSC_ARCH exists, otherwise >> use PETSC_DIR/include/petscconf.h . Perhpas I'm missing something, but >> this should be near to work. > > We've avoided using gnu make extensions - so there is no usage of 'if' > directives in petsc makefiles. [and we've used include directive as > alternative] >
I was not clear enough. I'm talking about a shell "if", something like if [-f ${PETSC_DIR}/${PETSC_ARCH}/include/petscconf.h]; then \ egrep ... else egrep .... fi -- Lisandro Dalcin --------------- CIMEC (INTEC/CONICET-UNL) Predio CONICET-Santa Fe Colectora RN 168 Km 472, Paraje El Pozo 3000 Santa Fe, Argentina Tel: +54-342-4511594 (ext 1011) Tel/Fax: +54-342-4511169