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

Allow use of "nc-config" as fall back if check_lib() fails #9

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

wwlwpd
Copy link
Contributor

@wwlwpd wwlwpd commented Jan 23, 2025

Issue 8: If check_lib() fails to find the netCDF libraries, then look for nc-config via PATH (using which); if found, use the command, nc-config --cflags to get the include flags for the linker (-I...). nc-config --libs returns the lib and lib path flags.

Resolves #8.

@wwlwpd
Copy link
Contributor Author

wwlwpd commented Jan 23, 2025

Hope this gets us most of the way there.

@oodler577 oodler577 mentioned this pull request Jan 23, 2025
Issue 8: If check_lib() fails to find the netCDF libraries, then
look for `nc-config` via PATH (using `which`); if found, use the
command, `nc-config --cflags` to get the include flags for the
linker (-I...). `nc-config --libs` returns the lib and lib path
flags.

Resolves PDLPorters#8.
@mohawk2
Copy link
Member

mohawk2 commented Jan 23, 2025

I've updated the main CI for the Brave New World of >=5.16, and will force-push this rebased past that once it finishes updating the caches (which is slow for some reason). If it then passes, which it looks like it will, I'll merge and release.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 12936462513

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 78.861%

Totals Coverage Status
Change from base Build 12936355273: 0.1%
Covered Lines: 720
Relevant Lines: 913

💛 - Coveralls

@mohawk2 mohawk2 merged commit 928a3b8 into PDLPorters:master Jan 23, 2025
6 checks passed
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.

support nc-config
3 participants