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

netcdf: add netcdf/4.7.4 recipe #3350

Merged
merged 5 commits into from
Jan 7, 2021

Conversation

madebr
Copy link
Contributor

@madebr madebr commented Oct 29, 2020

Specify library name and version: netcdf/4.7.4

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

@conan-center-bot
Copy link
Collaborator

Some configurations of 'netcdf/4.7.4' failed in build 1 (dbbb80c04124dc2cc788b1acf4232ca07bf6497a):

@conan-center-bot
Copy link
Collaborator

Some configurations of 'netcdf/4.7.4' failed in build 2 (ae5c56b3dfaa2dfc0d32d5af3db2b27ac34a8637):

@ghost
Copy link

ghost commented Nov 16, 2020

I detected other pull requests that are modifying netcdf/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

@ghost ghost mentioned this pull request Nov 16, 2020
4 tasks
@conan-center-bot
Copy link
Collaborator

All green in build 3 (7cdcb83261d0d0b37e18d912d0b0b878f3dd2d55)! 😊

recipes/netcdf/all/conanfile.py Outdated Show resolved Hide resolved
recipes/netcdf/all/conanfile.py Outdated Show resolved Hide resolved
@conan-center-bot
Copy link
Collaborator

All green in build 4 (faadb260a00e84790bf5fa8d9e28edd5ff875141)! 😊

@prince-chrismc
Copy link
Contributor

Please retrigger CI =)

@madebr
Copy link
Contributor Author

madebr commented Jan 4, 2021

@prince-chrismc
Thanks for the reminder

@conan-center-bot
Copy link
Collaborator

All green in build 6 (80e653ef483408391e2fc0176cd9ba5c296df0ee)! 😊

Copy link
Member

@uilianries uilianries left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

"cdf5": True,
"dap": True,
}
generators = "cmake_find_package", "cmake_find_package_multi", "cmake"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I rarely see both cmake_find_package and cmake_find_package_multi are needed

+ file(RELATIVE_PATH __makedist_cur_path "${PROJECT_SOURCE_DIR}" "${__makedist_full_cur_path}")
elseif(__makedist_extradist)
- file(RELATIVE_PATH __makedist_cur_path "${CMAKE_BINARY_DIR}" "${__makedist_full_cur_path}")
+ file(RELATIVE_PATH __makedist_cur_path "${PROJCT_BINARY_DIR}" "${__makedist_full_cur_path}")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
+ file(RELATIVE_PATH __makedist_cur_path "${PROJCT_BINARY_DIR}" "${__makedist_full_cur_path}")
+ file(RELATIVE_PATH __makedist_cur_path "${PROJECT_BINARY_DIR}" "${__makedist_full_cur_path}")

@conan-center-bot conan-center-bot merged commit 04873b0 into conan-io:master Jan 7, 2021
@Croydon
Copy link
Contributor

Croydon commented Jan 7, 2021

Oh damn. While going through issues I noticed this comment: #1850 (comment)

This probably should have been named netcdf-c instead. netcdf-cxx also exists.

@Croydon Croydon mentioned this pull request Jan 7, 2021
@d70-t
Copy link

d70-t commented Jan 7, 2021

Thank you all for finishing the netcdf recipe! I got sidetracked with my PR and am very happy that I don't have to worry about it anymore. 👍 💯

@madebr madebr deleted the netcdf_recipe branch January 7, 2021 15:00
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.

7 participants