On Sun, Nov 2, 2025 at 3:47 PM Piotr P. Karwasz <[email protected]> wrote: > > Hi Gary, > > On 2.11.2025 21:10, Gary Gregory wrote: > > This vote will close no sooner than 72 hours from now. > > > > [ ] +1 Release these artifacts > > [ ] +0 OK, but... > > [ ] -0 OK, but really should fix... > > [ ] -1 I oppose this release because... > > +1 (binding): release the artifacts. > > I've verified the following: > > - Signatures and checksums: All archives and Maven artifacts are valid. > - Unit tests: All tests pass successfully. > - RAT report: The `leaf.svg` file (an XML file) could include a license > header, but this is non-blocking.
Fixed in master. > - Japicmp report: All API changes are correctly listed in the changelog. > - Reproducibility: Maven artifacts reproduce identically with > - JDK 25 > - TZ=UTC > - Command used: > mvn verify artifact:compare \ > > -Dreference.repo=https://repository.apache.org/content/repositories/orgapachecommons-1870/ > \ > -Dbuildinfo.ignore="*/*spdx.json" > > Unlike other releases you've prepared, the `scm/tag` element in the POM > is set to `HEAD` instead of the release tag that will be used to mark > this version. I'm not even sure we should be using this element: Most components don't use it, and I don't use the Maven SCM plugin to create the tags (I use git). I'm inclined to remove this element. WDYT? Everything else looks good to me, thanks for preparing the > release! YW :) Gary > > Piotr > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
