Skip to content
New issue

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

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

Already on GitHub? # to your account

Convert the ENABLE_XXX options to NETCDF_ENABLE_XXX options for NCZarr #2923

Merged
merged 2 commits into from
May 13, 2024

Conversation

DennisHeimbigner
Copy link
Collaborator

Primary Change

In order to conform to the cmake overhaul, occurrences of ENABLE_XXX options in the nczarr code have been changed to NETCDF_ENABLE_XXX.

Misc. Other changes

  • Fix use of rand_s in libdispatch/ncrandom.c
  • Fix some bugs in the mingw gitub action.
  • Fix signature bug in libncpoco/cp_win32.c
  • Make some NCZarr fixes to config.h.cmake.in

# Primary Change
In order to conform to the cmake overhaul, occurrences of
ENABLE_XXX options in the nczarr code have been changed to
NETCDF_ENABLE_XXX.

# Misc. Other changes
* Fix use of rand_s in libdispatch/ncrandom.c
* Fix some bugs in the mingw gitub action.
* Fix signature bug in libncpoco/cp_win32.c
* Make some NCZarr fixes to config.h.cmake.in
@DennisHeimbigner DennisHeimbigner requested a review from WardF as a code owner May 11, 2024 22:24
@WardF WardF merged commit 2960f56 into Unidata:main May 13, 2024
103 checks passed
@DennisHeimbigner DennisHeimbigner deleted the optionchange.dmh branch May 16, 2024 00:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants