Remove TAO as stand-alone optional dependency and use via PETSc

Issue #267 resolved
Prof Garth Wells created an issue

Comments (11)

  1. Jan Blechta
    • changed status to open

    @cmaurini Now DOLFIN can't be build on PETSc release 3.4.4 because in CMakeLists.txt there is assumption that there is petsctao.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.

  2. Corrado Maurini

    Sorry, do you want me to submit a new pull request or may you implement the change directly?

  3. Prof Garth Wells reporter

    Could we have another go at sorting this one out? It would be good to get it into the 1.4 release.

  4. Corrado Maurini

    @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 with petsc-dev you need to set a cmake option -DDOLFIN_ENABLE_TAO = true. If you compile with the upcoming petsc-3.5 (probably 01/06 according to petsc mailing list), it will automatically compile with the TAO solver.

  5. Log in to comment