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

Update PIO from pio2_6_2 to pio2_6_5 #359

Merged
merged 2 commits into from
Feb 24, 2025
Merged

Update PIO from pio2_6_2 to pio2_6_5 #359

merged 2 commits into from
Feb 24, 2025

Conversation

billsacks
Copy link
Member

I did this update by simply copying the ParallelIO repository (checked
out at pio2_6_5) to here, and then deleting the .git directory. I
confirmed that, prior to this update, there were no diffs from the
ParallelIO repository (at pio2_6_2), so it is safe to do the update in
this simple manner.

Changes here:

  • pio2_6_3: Merge pull request #1982 from alperaltuntas/bugfix_debug: fix minor bugs in pio_nc4.c
  • pio2_6_4: Bug fix release. Fixes issues with large datasets and with autoconf/cmake configure option.
  • pio2_6_5: Updated for compatibility with netcdf-c 4.9.3 and cygwin.

Resolves #353

This reverts commit 1631dd8.

Reverting this in preparation for updating PIO version
I did this update by simply copying the ParallelIO repository (checked
out at pio2_6_5) to here, and then deleting the .git directory. I
confirmed that, prior to this update, there were no diffs from the
ParallelIO repository (at pio2_6_2), so it is safe to do the update in
this simple manner.

Changes here:

pio2_6_3: Merge pull request #1982 from alperaltuntas/bugfix_debug
- fix minor bugs in pio_nc4.c

pio2_6_4: Bug fix release. Fixes issues with large datasets and with
autoconf/cmake configure option.

pio2_6_5: Updated for compatibility with netcdf-c 4.9.3 and cygwin.

Resolves #353
@billsacks
Copy link
Member Author

@jedwards4b @theurich @oehmke - FYI

I have run testing on my Mac and all tests pass. It looks like, at least for some PIO updates in the past, we have run full nightly testing on a branch before merging. The changes here look fairly minimal, though, so my feeling is that that seems unnecessary here. @theurich or @oehmke - any feelings on whether to do that or just merge and check the regular nightly testing the next day?

@oehmke
Copy link
Contributor

oehmke commented Feb 24, 2025 via email

@billsacks billsacks merged commit 7f55bfb into develop Feb 24, 2025
6 checks passed
@billsacks billsacks deleted the pio265 branch February 24, 2025 21:59
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.

ESMF doesn't build with netcdf-c 4.9.3
2 participants