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: update to 4.9.3 #27638

Closed
wants to merge 1 commit into from
Closed

netcdf: update to 4.9.3 #27638

wants to merge 1 commit into from

Conversation

remkos
Copy link
Contributor

@remkos remkos commented Feb 12, 2025

Description

Update to upstream version 4.9.3.

With this release, all the patches for this port have become obsolete.

Since the library version number changed, all the dependencies have been revbumped, except those that failed (xmgr, py-nio, R-easyNCDF, R-ncdf4, R-Rnetcdf, aircraft_oap, h4h5tools, metview, ncarg, vapor).

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 15.3 24D60 x86_64
Command Line Tools 16.2.0.0.1.1733547573

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL in commit message?
  • checked your Portfile with port lint?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

@macportsbot
Copy link

Notifying maintainers:
@Dave-Allured for port nccmp, netcdf-fortran.
@dstrubbe for port paraview.
@Veence for port gdal.
@MarcusCalhoun-Lopez for port petsc, gerris.
@Schamschula for port octave-netcdf.
@jswhit for port py-netcdf4.
@petrrr for port py-nio.
@tenomoto for port cdo, ecCodes, esmf, gmt4, grads, magicspp, metview, ncarg, nco, netcdf-cxx, netcdf-cxx4, netcdf-fortran, netcdf, vis5d, wgrib2.
@jwhowarth for port pymol.
@barracuda156 for port R-ncdf4, elmerfem, mfem.
@nilason for port gdal.
@thiagoveloso for port ncview.
@jjstickel for port paraview.

@Schamschula
Copy link
Contributor

It looks like there is an issue building mpi variants. When I try to build with +mpich-default I get this error.

-- Found MPI_C: /opt/local/bin/mpicc-mpich-mp (found version "4.1")
-- Checking for module 'mpi-cxx'
--   No package 'mpi-cxx' found
-- Could NOT find MPI_CXX (missing: MPI_CXX_LIB_NAMES MPI_CXX_HEADER_DIR MPI_CXX_WORKS) 
CMake Error at /opt/local/share/cmake-3.31/Modules/FindPackageHandleStandardArgs.cmake:233 (message):
  Could NOT find MPI (missing: MPI_CXX_FOUND) (found version "4.1")
Call Stack (most recent call first):
  /opt/local/share/cmake-3.31/Modules/FindPackageHandleStandardArgs.cmake:603 (_FPHSA_FAILURE_MESSAGE)
  /opt/local/share/cmake-3.31/Modules/FindMPI.cmake:1842 (find_package_handle_standard_args)
  cmake/dependencies.cmake:486 (find_package)
  CMakeLists.txt:923 (include)


-- Configuring incomplete, errors occurred!

@barracuda156
Copy link
Contributor

For my ports: as long as CI pass for them, all should be good.

@remkos
Copy link
Contributor Author

remkos commented Feb 14, 2025

I'm giving up on this. Too many dependencies do not compile, and as @Schamschula reported, also the +mpich variant does not configure.

I will leave it to someone else to get this working.

@remkos remkos closed this Feb 14, 2025
@remkos remkos deleted the remkos-netcdf branch February 14, 2025 18:36
@Dave-Allured
Copy link
Contributor

MPI variants: Reported several times for older Netcdf versions.
https://trac.macports.org/ticket/66209 etc.

@Schamschula
Copy link
Contributor

Yes, it's been a problem for a while.

@nilason
Copy link
Contributor

nilason commented Feb 14, 2025

All ports succeeded with macOS 13, only one R-ncdf4 failed on macOS14. Without prebuilt packages for 15, it is not expected to succeed.
Seems to me to have been good to go!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.