@xylar can you review this PR?

This PR modifies the following package(s), for which you are listed as a maintainer:

• nco

We work closely with @czender, the main developer of NCO. We also use NCO frequently in our spack workflows and have been maintaining and using these versions on our own fork (https://github.com/E3SM-Project/spack). It seemed like the right thing to do to contribute back to the broader spack community.

@tldahlgren Thank you for the help! I just implemented the changes you suggested.

Thanks very much for the review, @tldahlgren!

@xylar I made those suggested changes and added 5.3.2.

@tldahlgren @alalazo, we're wondering what the way forward is here. It seems that CI failed 2 weeks ago on a dependency package, netcdf-c:
https://gitlab.spack.io/spack/spack/-/jobs/15219138

checking whether we are cross compiling... configure: error: in `/tmp/gitlab-runner-1/spack-stage/spack-stage-netcdf-c-4.9.2-dlnjfxshv47hzz7ekcvo4yvptdbne75h/spack-src':
configure: error: cannot run C compiled programs.

This would appear to be an issue that might require attention from the maintainers of that package or the spack team but it doesn't seem to be directly related to nco.

What are your thoughts?

Thanks @tldahlgren. Good thought! Maybe recent changes will fix that issue.

Thanks @tldahlgren. Good thought! Maybe recent changes will fix that issue.

🤞

Thanks @tldahlgren!