Skip to content

Release 5.9.0 - #1472

Merged
lesserwhirls merged 3 commits into
maint-5.xfrom
5.9.0
Aug 8, 2025
Merged

lesserwhirls merged 3 commits into
maint-5.xfrom
5.9.0

Conversation

@lesserwhirls

Copy link
Copy Markdown
Member

Release v5.9.0

@lesserwhirls
lesserwhirls marked this pull request as ready for review August 8, 2025 16:11
@lesserwhirls
lesserwhirls merged commit ad7903a into maint-5.x Aug 8, 2025
20 checks passed
@lesserwhirls
lesserwhirls deleted the 5.9.0 branch August 8, 2025 16:11
@rschmunk

rschmunk commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

I noticed a few days ago with a snapshot and also see this now when building 5.9.0 from scratch that the UDUNITS section is showing some warnings. Perhaps due to #1461?

`

Task :udunits:compileJavacc
Java Compiler Compiler Version 7.0.13 (Parser Generator)
(type "javacc" with no arguments for help)
Reading from file /some/path/netcdf-java-5.9.0/udunits/src/main/javacc/StandardUnitFormat.jj . . .
Warning: Choice conflict in (...)* construct at line 562, column 5.
Expansion nested within construct and expansion following construct
have common prefixes, one of which is:
Consider using a lookahead of 2 or more for nested expansion.
Warning: Choice conflict in [...] construct at line 570, column 13.
Expansion nested within construct and expansion following construct
have common prefixes, one of which is: "."
Consider using a lookahead of 2 or more for nested expansion.
Warning: Choice conflict in [...] construct at line 594, column 5.
Expansion nested within construct and expansion following construct
have common prefixes, one of which is: "+"
Consider using a lookahead of 2 or more for nested expansion.
`

@lesserwhirls

Copy link
Copy Markdown
Member Author

That's exactly it. I'm almost willing to bet those warning where there the last time these files were generated and committed to the project (in 2009). Now that we actually see them, we have a chance to address them.

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.

2 participants