solr icon indicating copy to clipboard operation
solr copied to clipboard

SOLR-15910: Fork semver4j library to fix semver bugs.

Open HoustonPutman opened this issue 3 years ago • 5 comments

https://issues.apache.org/jira/browse/SOLR-15910

HoustonPutman avatar Jan 11 '22 22:01 HoustonPutman

This is kind of a pain, but I’d like to see the commits split into a clean fork and then our changes separately. Also, I don’t think we repackage the files, see org.noggit

madrob avatar Jan 11 '22 22:01 madrob

There are already 11 contributors to https://github.com/vdurmont/semver4j. Have you tried contacting them? Perhspa if we submit a PR for https://github.com/vdurmont/semver4j/issues/48 and ping a few of the devs, we can help trigger a new release? Two years old can also mean that it is mature and don't need that many updates...

janhoy avatar Jan 11 '22 23:01 janhoy

Maybe for an adjacent library, just having a fixed version that lives as someone's personal fork in Github would also be okay? Versus bringing in this code to the Solr project itself.

Github makes it too easy to get lots of forks that all end up vaguely unmaintained, and I wish it encouraged a cluster of forks like what we see for semver to somehow come back together to a single maintained approach. Oh wait... community over code ;-)

epugh avatar Jan 12 '22 14:01 epugh

Completely agree with both of you that not forking would be the best path forward. It's hard company-policy-wise for me to move forward with either of those options, but I would appreciate either of you taking it up!

HoustonPutman avatar Jan 13 '22 16:01 HoustonPutman

I'll not veto the code copy, if you clearly mark it and open, say, a blocker JIRA for 10.0 or something to replace it with something that is maintained...

janhoy avatar Jan 14 '22 13:01 janhoy

Looks like this was fixed and forked - https://github.com/semver4j/semver4j potentially?

risdenk avatar Oct 14 '22 21:10 risdenk

I updated this PR with my changes. I realized I didn't need a new PR since I was able to merge main and then use semver4j.

I need to fix a few things still solr/CHANGES.txt and maybe some other minor things, but overall this is ready to go.

risdenk avatar Oct 15 '22 00:10 risdenk

:warning: 314 God Classes were detected by Lift in this project. Visit the Lift web console for more details.

sonatype-lift[bot] avatar Oct 15 '22 01:10 sonatype-lift[bot]