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

Use FindPkgConfig and petsc.pc instead of FindPETSc.cmake closes #1913 #1932

Merged

Conversation

prudhomm
Copy link
Member

@prudhomm prudhomm commented Sep 9, 2022

  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes?
  • Have you successfully run the Feel++ testsuite with your changes locally?
  • Have you written Doxygen comments in your contribution ?

@prudhomm prudhomm added this to the v110 milestone Sep 9, 2022
@prudhomm prudhomm self-assigned this Sep 9, 2022
@prudhomm prudhomm linked an issue Sep 9, 2022 that may be closed by this pull request
@prudhomm
Copy link
Member Author

to do still

prudhomm and others added 11 commits September 11, 2022 10:59
#1913 and Port Feel++  to Fedora 36/37 #1914
…dora #1934

- update pybind11 to latest release
- add option to use system installation
update pybind11 to latest version to support python3.11 on Fedora #1934
Port Feel++  to Fedora 36/37 #1914
 _1 not found need std::placeholder but use lambda instead for readbility

Port Feel++  to Fedora 36/37 #1914
the code was a no-op

Port Feel++  to Fedora 36/37 #1914
@prudhomm
Copy link
Member Author

@vincentchabannes feel++ compiles on fedora:37 (I haven't tested yet on fedora:36), several compilation fixes have been made due to clang 14.
Also the documentation of the toolbox applications have been updated and are now generated automatically

Port Feel++  to Fedora 36/37 #1914
Port Feel++  to Fedora 36/37 #1914
@prudhomm
Copy link
Member Author

@vincentchabannes the PR builds fine. The remaining builds are due to changes to build fedora via github actions. The important build is this one: https://buildkite.com/feelpp/feelpp/builds/4655.
Please review

@prudhomm
Copy link
Member Author

the current build is here : https://buildkite.com/feelpp/feelpp/builds/4668

@prudhomm prudhomm merged commit c2fc489 into develop Oct 31, 2022
@prudhomm prudhomm deleted the 1913-use-findpkgconfig-and-petscpc-instead-of-findpetsccmake branch October 31, 2022 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Port Feel++ to Fedora 36/37 Use FindPkgConfig and petsc.pc instead of FindPETSc.cmake
2 participants