<div dir="ltr"><div dir="ltr">On Fri, Jun 28, 2019 at 2:04 PM Smith, Barry F. via petsc-dev <<a href="mailto:petsc-dev@mcs.anl.gov">petsc-dev@mcs.anl.gov</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
  You are right, these do not belong in petscconf.h<br></blockquote><div><br></div><div>The problematic thing here is hiding information from users of PETSc. If you are a user that</div><div>counts on PETSc configure to check something, but then we hide it because we do not use it,</div><div>I would not be happy.</div><div><br></div><div>    Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
 Barry<br>
<br>
<br>
> On Jun 28, 2019, at 12:37 PM, Jed Brown via petsc-dev <<a href="mailto:petsc-dev@mcs.anl.gov" target="_blank">petsc-dev@mcs.anl.gov</a>> wrote:<br>
> <br>
> We have a lot of lines like this<br>
> <br>
> $ grep -c HAVE_LIB $PETSC_ARCH/include/petscconf.h<br>
> 96<br>
> <br>
> but only four of these are ever checked in src/.  Delete them?<br>
> <br>
> IMO, unused stuff should not go into petscconf.h.  We have to scroll up<br>
> past these lines every time configure crashes.  These are apparently all<br>
> unused:<br>
> <br>
> $ for name in $(grep '#define \w\+ ' $PETSC_ARCH/include/petscconf.h | cut -f 2 -d\ ); do rg -q -c $name include/ src/ || echo $name; done<br>
> STDC_HEADERS<br>
> HAVE_MATH_INFINITY<br>
> ANSI_DECLARATORS<br>
> PETSC_PATH_SEPARATOR<br>
> PETSC_HAVE_BLASLAPACK<br>
> PETSC_HAVE_PTHREAD<br>
> PETSC_HAVE_NETCDF<br>
> PETSC_HAVE_PNETCDF<br>
> PETSC_HAVE_METIS<br>
> PETSC_HAVE_MATHLIB<br>
> PETSC_HAVE_ZLIB<br>
> PETSC_TIME_WITH_SYS_TIME<br>
> PETSC_HAVE_MATH_H<br>
> PETSC_HAVE_ENDIAN_H<br>
> PETSC_HAVE_LIMITS_H<br>
> PETSC_HAVE_SEARCH_H<br>
> PETSC_HAVE_SCHED_H<br>
> PETSC_HAVE_PTHREAD_H<br>
> PETSC_HAVE_CXX_NAMESPACE<br>
> PETSC_HAVE_FORTRAN_TYPE_INITIALIZE<br>
> PETSC_HAVE_LIBDL<br>
> PETSC_HAVE_LIBQUADMATH<br>
> PETSC_HAVE_LIBX11<br>
> PETSC_HAVE_LIBZ<br>
> PETSC_HAVE_LIBTRIANGLE<br>
> PETSC_HAVE_LIBMETIS<br>
> PETSC_HAVE_LIBPARMETIS<br>
> PETSC_HAVE_LIBHDF5HL_FORTRAN<br>
> PETSC_HAVE_LIBHDF5_FORTRAN<br>
> PETSC_HAVE_LIBHDF5_HL<br>
> PETSC_HAVE_LIBHDF5<br>
> PETSC_HAVE_LIBPNETCDF<br>
> PETSC_HAVE_LIBNETCDF<br>
> PETSC_HAVE_LIBEXODUS<br>
> PETSC_HAVE_LIBBLAS<br>
> PETSC_HAVE_LIBLAPACK<br>
> PETSC_HAVE_LIBSTDC__<br>
> PETSC_HAVE_LIBMPI_USEMPIF08<br>
> PETSC_HAVE_LIBMPI_USEMPI_IGNORE_TKR<br>
> PETSC_HAVE_LIBMPI_MPIFH<br>
> PETSC_HAVE_LIBMPI<br>
> PETSC_HAVE_LIBGFORTRAN<br>
> PETSC_HAVE_LIBGCC_S<br>
> PETSC_HAVE_LIBPTHREAD<br>
> PETSC_HAVE_LIBSUNDIALS_CVODE<br>
> PETSC_HAVE_LIBSUNDIALS_NVECSERIAL<br>
> PETSC_HAVE_LIBSUNDIALS_NVECPARALLEL<br>
> PETSC_HAVE_LIBML<br>
> PETSC_HAVE_LIBSUPERLU_DIST<br>
> PETSC_HAVE_LIBSUPERLU<br>
> PETSC_HAVE_LIBUMFPACK<br>
> PETSC_HAVE_LIBKLU<br>
> PETSC_HAVE_LIBCHOLMOD<br>
> PETSC_HAVE_LIBBTF<br>
> PETSC_HAVE_LIBCCOLAMD<br>
> PETSC_HAVE_LIBCOLAMD<br>
> PETSC_HAVE_LIBCAMD<br>
> PETSC_HAVE_LIBAMD<br>
> PETSC_HAVE_LIBSUITESPARSECONFIG<br>
> PETSC_HAVE_LIBSCALAPACK<br>
> PETSC_HAVE_LIBCMUMPS<br>
> PETSC_HAVE_LIBDMUMPS<br>
> PETSC_HAVE_LIBSMUMPS<br>
> PETSC_HAVE_LIBZMUMPS<br>
> PETSC_HAVE_LIBMUMPS_COMMON<br>
> PETSC_HAVE_LIBPORD<br>
> PETSC_HAVE_LIBHYPRE<br>
> PETSC_USE_SCALAR_REAL<br>
> PETSC_HAVE_REAL___FLOAT128<br>
> PETSC_RETSIGTYPE<br>
> PETSC_SIZEOF_SHORT<br>
> PETSC_SIZEOF_MPI_COMM<br>
> PETSC_SIZEOF_MPI_FINT<br>
> PETSC_HAVE_VPRINTF<br>
> PETSC_HAVE_VFPRINTF<br>
> PETSC_HAVE_MKSTEMP<br>
> PETSC_HAVE_GETHOSTBYNAME<br>
> PETSC_HAVE_SOCKET<br>
> PETSC_HAVE_TIMES<br>
> PETSC_HAVE_SIGSET<br>
> PETSC_HAVE_GETTIMEOFDAY<br>
> PETSC_HAVE_SIGNAL<br>
> PETSC_HAVE_GET_NPROCS<br>
> PETSC_HAVE_SIGACTION<br>
> PETSC_HAVE__GFORTRAN_IARGC<br>
> PETSC_HAVE_SHARED_LIBRARIES<br>
> PETSC_USE_GDB_DEBUGGER<br>
> PETSC_VERSION_BRANCH_GIT<br>
> PETSC_HAVE_MPI_COMM_F2C<br>
> PETSC_HAVE_MPI_COMM_C2F<br>
> PETSC_HAVE_MPI_FINT<br>
> PETSC_HAVE_MPI_REPLACE<br>
> PETSC_HAVE_MPI_WIN_ALLOCATE_SHARED<br>
> PETSC_HAVE_MPI_WIN_SHARED_QUERY<br>
> PETSC_HAVE_MPI_ALLTOALLW<br>
> PETSC_HAVE_MPI_COMM_SPAWN<br>
> PETSC_HAVE_MPI_TYPE_GET_EXTENT<br>
> PETSC_HAVE_PTHREAD_BARRIER_T<br>
> PETSC_HAVE_SCHED_CPU_SET_T<br>
> PETSC_HAVE_SYS_SYSCTL_H<br>
> PETSC_LEVEL1_DCACHE_SIZE<br>
> PETSC_LEVEL1_DCACHE_ASSOC<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>-- Norbert Wiener</div><div><br></div><div><a href="http://www.cse.buffalo.edu/~knepley/" target="_blank">https://www.cse.buffalo.edu/~knepley/</a><br></div></div></div></div></div></div></div></div>