Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make PETSc and petsc4py required dependencies #73

Closed
chrisrichardson opened this issue May 10, 2018 · 6 comments
Closed

Make PETSc and petsc4py required dependencies #73

chrisrichardson opened this issue May 10, 2018 · 6 comments

Comments

@chrisrichardson
Copy link
Contributor

Make PETSc and petsc4py both required dependencies in CMakeLists.txt

@jhale
Copy link
Member

jhale commented Jul 6, 2018

Fixed in #118

@chrisrichardson
Copy link
Contributor Author

Actually, I think this was slightly wrong. petsc4py should not be a required dependency of the C++ build, only the Python build.

@jhale
Copy link
Member

jhale commented Jan 21, 2019

Looking at master, that is how it is.

@garth-wells
Copy link
Member

The C++ build system should be fixed so it does not depend on petsc4py. All the necessary info is available via the PETSc headers.

@chrisrichardson
Copy link
Contributor Author

chrisrichardson commented Jan 21, 2019

-- Generating form files in demo and test and directories. May take some time...
-- ----------------------------------------------------------------------------------------
CMake Error at CMakeLists.txt:343 (message):
  Generation of form files failed:

  Traceback (most recent call last):

    File "/tmp/dolfinx/cpp/cmake/scripts/generate-form-files.py", line 21, in <module>
      from petsc4py import PETSc

  ModuleNotFoundError: No module named 'petsc4py'

@garth-wells
Copy link
Member

We should get generate-form-files.py to take command lines arguments that are passed through to ffc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants