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

pkg-config petsc (next) #1914

Merged
merged 6 commits into from
Apr 7, 2020
Merged

pkg-config petsc (next) #1914

merged 6 commits into from
Apr 7, 2020

Conversation

dschwoerer
Copy link
Contributor

Allow to use pkg-config to detect petsc.
PETSc provides pkg-config files. Unlike the normal method, this is less dependent on finding the magic files of petsc in various locations, and can be compatible with the FHS

#1913 for next

…-next

 Conflicts:
	src/fileio/datafile.cxx

Use master version, where we do not add the same variable twice.
@ZedThree
Copy link
Member

Looks like this has pulled in a bunch of commits from elsewhere.

@dschwoerer
Copy link
Contributor Author

I merged master into next, as master contains e.g. a bug fix, that is not fixed in next:

5403eeb:

 Conflicts:
	src/fileio/datafile.cxx

Use master version, where we do not add the same variable twice.

@dschwoerer
Copy link
Contributor Author

Replaced by #1999

@dschwoerer dschwoerer closed this Mar 27, 2020
@dschwoerer dschwoerer reopened this Apr 4, 2020
@dschwoerer
Copy link
Contributor Author

Something went wrong with #1999

The configure.ac master history list the pkg-config change:
https://github.com/boutproject/BOUT-dev/commits/master/configure.ac

However, the next one doesn't:
https://github.com/boutproject/BOUT-dev/commits/next/configure.ac

If we cannot find PETSc with PETSC_DIR/PETSC_ARCH, we ask pkg-conf
to determine PETSc version and required flags.
@dschwoerer
Copy link
Contributor Author

I had to cherry-pick the old commits. The CMake changes where still there, but the configure.ac changes where gone ...

@dschwoerer dschwoerer requested a review from ZedThree April 4, 2020 00:25
@dschwoerer
Copy link
Contributor Author

@ZedThree Can you please merge this?
It was in, and was removed by accident.

If you would like me to clean the history, let me know.

@ZedThree ZedThree merged commit c7397a3 into next Apr 7, 2020
@ZedThree ZedThree deleted the pkg-config-petsc-next branch April 7, 2020 08:53
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

Successfully merging this pull request may close these issues.

None yet

2 participants