Skip to content

Conversation

pjfanning
Copy link
Member

@pjfanning pjfanning commented Jun 9, 2025

@pjfanning pjfanning marked this pull request as draft June 9, 2025 19:28
@cowtowncoder
Copy link
Member

It seems like we should probably only include LTS-matched classes, so leave JDK 21 but drop 23.

@pjfanning
Copy link
Member Author

This is currently blocked. maven-shade-plugin is omitting some new subpackages from FastDoubleParser. I have tried a few changes to the config setup but the classes are still missing so far.

@pjfanning pjfanning marked this pull request as ready for review June 10, 2025 09:55
@pjfanning
Copy link
Member Author

The issue seems that the <minimizeJar>true</minimizeJar> seems to lead to a lot of the FastDoubleParser classes being removed, presumably because we don't need them.

We could merge this and if anything breaks in other Jackson builds, we could revert this.

pom.xml Outdated
@@ -175,6 +175,7 @@ com.fasterxml.jackson.core.*;version=${project.version}
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
<version>3.6.0</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not needed, should come from "oss-parent", will remove.

@cowtowncoder cowtowncoder changed the title fastdoubleparser 2.0.1 Upgrade FastDoubleParser to 2.0.1 Jun 11, 2025
@cowtowncoder
Copy link
Member

Sure, I can merge this now and we can see what (if anything) happens downstream.

@cowtowncoder cowtowncoder merged commit 69423a1 into FasterXML:2.x Jun 12, 2025
7 checks passed
@cowtowncoder cowtowncoder added this to the 2.20.0 milestone Jun 12, 2025
@pjfanning pjfanning deleted the fdp2 branch June 12, 2025 09:09
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