[petsc-dev] (no subject)
Jed Brown
jedbrown at mcs.anl.gov
Sat Feb 16 18:20:01 CST 2013
On Sat, Feb 16, 2013 at 6:00 PM, Barry Smith <bsmith at mcs.anl.gov> wrote:
> No, as I said in my previous email, it would be in
> petsc-private/petscconfhypre.h
>
> When I wrote petsconfPACKAGE.h I meant this in the sense of
> PETSc/packages/*.py NOT in the sense of sys, ksp, vec, etc. so there would
> be a petsconfhypre.py petscconfml.py petscconfsuperlu.py etc etc etc
>
Ah, okay.
> Yes there would also be a petscconfsys.h for things needed in the the sys
> directory (just so they are not included in snes, ts, etc.
>
Just splitting petscconfsys.h and petscconfpackages.h might be enough
(leaving petscconfbase.h or whatever to contain things like
PETSC_USE_DEBUG) would remove dependencies from most source files (that
care about neither packages nor sys).
If we did this, we should at least put a comment /* #undef
PETSC_HAVE_PACKAGE */ for those missing packages, so that we can easily
grep to see which header the macro would be defined in.
> No, I am not strongly advocating making this change. Just suggesting it
> as a natural possibility related to the other recent changes
>
Yeah, I'm not convinced it's worth it.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mcs.anl.gov/pipermail/petsc-dev/attachments/20130216/6f870ae4/attachment.html>
More information about the petsc-dev
mailing list