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

release 4.0.5 to maven central (or another publicly accessible repo) #191

Open
afeinberg opened this issue Dec 12, 2015 · 7 comments
Open

Comments

@afeinberg
Copy link

4.0.5 contains a fix for qsappend (issue 116), but it isn't available anywhere. can 4.0.5 be pushed to a public repo?

@wbazant
Copy link

wbazant commented Jul 5, 2017

@paultuckey are you the only maintainer of this repository? Is there anyone else allowed to do releases of new versions?

@paultuckey
Copy link
Owner

sure happy for someone to do it. the only reason I haven't is that I haven't had the time to setup many app servers and run the full test suite over them all.

@tubbynl
Copy link

tubbynl commented Oct 16, 2017

@paultuckey i do think it's sensible to setup the release proces anyway

for running the full testsuite we could start with a single application server and work our way through. i do think that in a minimal setting being able to release is valuable on it's own.

either way extra maintainers ( @wbazant and/or @afeinberg ?) will be required to setup some sort of CI if @paultuckey doesn't have the time.

@chadlwilson
Copy link

chadlwilson commented Oct 18, 2023

Recently 5.x versions have been released which include migration to the jakarta.x namespaces - thanks for the work on this @paultuckey !

However there seem to be a number of changes/fixes that were never released for the 4.x stream over many years, including #116 mentioned here, but also #182.

If a build/release process is now set up, would it be possible to get a 4.x release with these fixes, given the 5.x change is a big breaking change that typically requires synchronised migration of all sorts of frameworks/libraries etc (web server/servlet container, web framework and more)?

Due to the lack of git tags for 5.x, it's a bit difficult to find appropriate start and end points but the changes up until when the POM was changed to 5.0.0-SNAPSOT seem to be urlrewritefilter-4.0.4...4d9d29d

There were a number of changes/fixes subsequent to the POM major version change in 2017 that seem to be relevant to 4.x as well, and perhaps could be cherry-picked.

@paultuckey
Copy link
Owner

Yep, I've now got a release process working ok. If anyone wants to submit a PR backporting changes to 4.x I'm happy to accept it. I can just make a long lived branch for 4.x.

@chadlwilson
Copy link

That would be great (to pick an appropriate 4.x branch starting point from your perspective)! I imagine at a minimum some of the more Maven-oriented build/release and POM changes will definitely be needed to be cherrypicked to allow easy release.

I haven't really dug through to understand which of the changes after ffd9159 were intentionally "breaking" as a v5 vs not-necessarily-breaking, as when the POM was initially bumped it didn't seem at the time motivated by a specific breaking change - more forward looking.

Of the PRs merged, if it's literally only the jakarta.servlet plus servlet-api 5.x changes that are truly breaking in 5.x perhaps that's easier because one can probably cherry pick basically everything except that change.

@paultuckey
Copy link
Owner

Yes that is a good commit to use. I think the changes related to releasing will be easy to copy in. The changes around container testing will not, so I'd probably skip those.

https://github.com/paultuckey/urlrewritefilter/tree/4.x

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

No branches or pull requests

5 participants