- changed status to open
Remove TAO as stand-alone optional dependency and use via PETSc
See discussion at http://fenicsproject.org/pipermail/fenics/2014-March/001335.html.
Comments (11)
-
-
@blechta I was just checking this. This pull-request should fix the issue:
https://bitbucket.org/fenics-project/dolfin/pull-request/129/fix-issue-267/diff
-
reporter @blechta Can you test Corrado's change? If it works for you I'll merge it.
-
@garth-wells It works but proposed changeset contains messy merges...
-
Sorry, do you want me to submit a new pull request or may you implement the change directly?
-
@cmaurini Non fast-forward merging without a purpose is not good, see PETSc git workflow which applies to DOLFIN also. It would be logical to continue fixing it where you finished but without merging
next
. -
@blechta After 20 minutes trying to understand git, I gave up and submitted a clean pull-request )-:
I hope this is fine now.
-
reporter Could we have another go at sorting this one out? It would be good to get it into the 1.4 release.
-
@garth-wells I think this is fine now. What do you want exactly? Now with
# Enable TAO from PETSc for PETSc verion > 3.4 if ((PETSC_VERSION VERSION_GREATER 3.4.100) OR (DOLFIN_ENABLE_TAO)) set(PETSC_ENABLE_TAO TRUE) endif()
in
CMakeList.txt
if you compile withpetsc-dev
you need to set a cmake option-DDOLFIN_ENABLE_TAO = true
. If you compile with the upcomingpetsc-3.5
(probably 01/06 according to petsc mailing list), it will automatically compile with the TAO solver. -
reporter - changed status to resolved
-
reporter - removed milestone
Removing milestone: 1.4 (automated comment)
- Log in to comment
@cmaurini Now DOLFIN can't be build on PETSc release 3.4.4 because in
CMakeLists.txt
there is assumption that there ispetsctao.h
in PETSc 3.4.4. It can't overriden by-DDOLFIN_ENABLE_TAO:BOOL=OFF
or-DPETSC_ENABLE_TAO:BOOL=OFF
,-DENABLE_PETSC_TAO:BOOL=OFF
.