<div dir="ltr"><div><div>Thanks Satish and Lisandro. It not a problem at all because PETSc is usable but such an if would avoid that message.<br><br></div>Regarding the compilation, installation and test instruction printed a screen, maybe also there it would be nice to have some if statements.<br>
<br></div>Thank you again.<br><div><div><br><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 15, 2013 at 10:32 AM, Satish Balay <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, 15 May 2013, Lisandro Dalcin wrote:<br>
<br>
> On 15 May 2013 17:12, Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> wrote:<br>
> > On Wed, 15 May 2013, Lisandro Dalcin wrote:<br>
> ><br>
> >> On 15 May 2013 16:54, Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> wrote:<br>
> >> > On Wed, 15 May 2013, Matteo Parsani wrote:<br>
> >> ><br>
> >> >> Hello Satish,<br>
> >> >> Lisandro has found that in the make file we have<br>
> >> >><br>
> >> >> include ././${PETSC_ARCH}/conf/petscvariables<br>
> >> ><br>
> >> > we try to support too many ways of installing petsc [with prefix/<br>
> >> > without prefix, with a defaut PETSC_ARCH, without PETSC_DIR set etc..]<br>
> >> > And we need this line for users who forget to set PETSC_ARCH in a<br>
> >> > non-prefix build. [to pick up a defaut PETSC_ARCH]<br>
> >> ><br>
> >><br>
> >><br>
> >> Satish, I think all what is needed is to fix a little the test_build<br>
> >> target. Basically, test if the file under PETSC_ARCH exists, otherwise<br>
> >> use PETSC_DIR/include/petscconf.h . Perhpas I'm missing something, but<br>
> >> this should be near to work.<br>
> ><br>
> > We've avoided using gnu make extensions - so there is no usage of 'if'<br>
> > directives in petsc makefiles. [and we've used include directive as<br>
> > alternative]<br>
> ><br>
><br>
> I was not clear enough. I'm talking about a shell "if", something like<br>
><br>
> if [-f ${PETSC_DIR}/${PETSC_ARCH}/include/petscconf.h]; then \<br>
>    egrep ...<br>
> else<br>
>    egrep ....<br>
> fi<br>
<br>
</div></div>Sure - I was hoping for a: 'if [ "${PREFIX}" = "" ]' - but currently<br>
configure doesn't appear so set such a thing. We could add that - or<br>
just use the if statement of petscconf.h as you suggest.<br>
<br>
Satish<br>
<br>
-----<br>
<br>
diff --git a/makefile b/makefile<br>
index 372e593..9bc5cfb 100644<br>
--- a/makefile<br>
+++ b/makefile<br>
@@ -138,7 +138,11 @@ test_build:<br>
        @cd src/snes/examples/tutorials >/dev/null; ${OMAKE} PETSC_ARCH=${PETSC_ARCH}  PETSC_DIR=${PETSC_DIR} clean<br>
        @cd src/snes/examples/tutorials >/dev/null; ${OMAKE} PETSC_ARCH=${PETSC_ARCH}  PETSC_DIR=${PETSC_DIR} testex19<br>
<div class="im">        @if [ "${FC}" != "" ]; then \<br>
</div>-          egrep "^#define PETSC_USE_FORTRAN_DATATYPES 1" ${PETSC_DIR}/${PETSC_ARCH}/include/petscconf.h | tee .ftn-dtype.log > /dev/null; \<br>
+          if [ -f ${PETSC_DIR}/${PETSC_ARCH}/include/petscconf.h ]; then \<br>
+            egrep "^#define PETSC_USE_FORTRAN_DATATYPES 1" ${PETSC_DIR}/${PETSC_ARCH}/include/petscconf.h | tee .ftn-dtype.log > /dev/null; \<br>
+          else \<br>
+            egrep "^#define PETSC_USE_FORTRAN_DATATYPES 1" ${PETSC_DIR}/include/petscconf.h | tee .ftn-dtype.log > /dev/null; \<br>
+          fi; \<br>
<div class="HOEnZb"><div class="h5">           if test -s .ftn-dtype.log; then F90TEST="testex5f90t"; else F90TEST="testex5f"; fi; ${RM} .ftn-dtype.log; \<br>
           cd src/snes/examples/tutorials >/dev/null; ${OMAKE} PETSC_ARCH=${PETSC_ARCH}  PETSC_DIR=${PETSC_DIR} $${F90TEST}; \<br>
          fi;<br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr">Matteo<br></div>
</div>