On Fri, Jan 10, 2020 at 9:16 AM William Speirs <wspe...@apache.org> wrote:

> For what it's worth, I'm unable to find that branch in git:
>
> $ git clone https://gitbox.apache.org/repos/asf/commons-dbutils.git
> --branch <https://gitbox.apache.org/repos/asf/commons-dbutils.git--branch>
> commons-dbutils-1.8-RC2 commons-dbutils-1.8-RC2
> Cloning into 'commons-dbutils-1.8-RC2'...
> fatal: Remote branch commons-dbutils-1.8-RC2 not found in upstream origin
>
> After cloning the repo, listing remote branches doesn't show anything:
>
> $ git branch -r
>   origin/2_0
>   origin/HEAD -> origin/master
>   origin/master
>   origin/release
>
> Bill-
>

Right, I think we need to hit the pause button here and sort out 1.x versus
a desired 2.0.

Since there is a what looks like a 2.0 branch and our current release is
1.x, I would assume that this release vote is being made from the release
branch which has been merged from master.

But that does not seem to be the case, right? Since the pom.xml in master
is 2.0-SNAPSHOT that would seem to indicate that the release branch comes
from elsewhere, a 1.x branch? No, we do not have one.

It feels to me like we should have one of either:

- branch master is 1.8-SNAPSHOT, merged into branch release. The 2.0 work
should be in branch 2_0 (which should renamed to 2.x IMO) until it is
ready, package name change and all.
or:
- branch master is 2.0-SNAPSHOT. A currently non-existing branch 1.x would
contain 1.x and branch release would come a merge of branch 1.x.

What do others think? The waters are too muddy for my scuba capability ATM
;-)

Gary


>
> On Fri, Jan 10, 2020 at 3:56 AM Bruno P. Kinoshita <ki...@apache.org>
> wrote:
>
> >     [x] +1 Release these artifacts
> >
> >
> > Build passing with JDK 13 on Ubuntu LTS, from git tag (mvn clean test
> > install site). Reports look OK.
> >
> >
> > There are two SpotBugs bugs about default encoding that can be fixed
> later
> > I suppose. Generating reports locally skipped PMD. For some reason, even
> > though the dist area's site has a PMD, and we have a pmd file in the
> root,
> > my generated site didn't include that report. Not a blocker.
> >
> >
> > Found no issues with signatures of files in the dist area or in this mvn
> > repo [1]. Inspected src zip, and also found no issues.
> >
> >
> > Thanks!
> >
> > Bruno
> >
> >
> > [1]
> >
> https://repository.apache.org/content/groups/staging/commons-dbutils/commons-dbutils/1.8/
> >
> >
> >     On Thursday, 9 January 2020, 7:50:40 pm NZDT, Carl Hall <
> > thecarlh...@apache.org> wrote:
> >
> >  We have fixed quite a few bugs and added some significant enhancements
> > since Apache Commons DbUtils 1.7 was released, so I would like to release
> > Apache Commons DbUtils 1.8.
> >
> > RC2 handles closing connections only when owned, and addresses generated
> > javadoc, NOTICE year update, and release notes detail.
> >
> > Apache Commons DbUtils 1.8 RC2 is available for review here:
> >     https://dist.apache.org/repos/dist/dev/commons/dbutils/1.8-RC2 (svn
> > revision 37533)
> >
> > The Git tag commons-dbutils-1.8-RC2 commit for this RC is
> > 9be04e5cc990deee3ba672aa8060c523db897b7a which you can browse here:
> >
> >
> https://gitbox.apache.org/repos/asf?p=commons-dbutils.git;a=commit;h=9be04e5cc990deee3ba672aa8060c523db897b7a
> >
> > You may checkout this tag using:
> >     git clone https://gitbox.apache.org/repos/asf/commons-dbutils.git
> > --branch commons-dbutils-1.8-RC2 commons-dbutils-1.8-RC2
> >
> > Maven artifacts are here:
> >
> >
> https://repository.apache.org/service/local/repositories/orgapachecommons-1488/content/commons-dbutils/commons-dbutils/1.8/
> >
> > These are the artifacts and their hashes:
> >
> > #Release SHA-512s
> > #Wed Jan 08 22:43:42 PST 2020
> >
> >
> commons-dbutils-1.8-tests.jar=0228b8f564642709b0581434f39433f872c77891b5a5d70f473aa8e6cfb14721890982fc4afcde064795ca63753d50ecfc2a173665174bf6fc2683af9770eed9
> >
> >
> commons-dbutils-1.8-bin.tar.gz.asc=ac5b2da84f1ca0f4605b61583c7d907cb68774a229620d08e7c36e4513435ffbebeb31cae313411e6b0f9dd70e60d55317826e14a43c7c47dcf1c98db03f2396
> >
> >
> commons-dbutils-1.8-test-sources.jar=075ac4a74cad06a34d901c088f323c59dbe1b9eea076404212ed63b7eb6fa3a266ada20aa63264e89c4203d8d6140fb556acd89ff4e08b21f3f9693de11543a5
> >
> >
> commons-dbutils-1.8-test-sources.jar.asc=baf8d0e9477d02eac7c3f4be5ac9358378bf46778b4c3159586b336e9300c18888ae8c89dd80b22daaeb6b0aafe09c3a6277baddef9e8898c990986cc57df989
> >
> >
> commons-dbutils-1.8.pom.asc=659f6bfda15f588d00bd53b4204ab2de8eba20a5e484619a1770816e9cb7372846499bc87b9b7629a3fc0c6e3b4f1b740362468aea232286318feeddc4c3bfc5
> >
> >
> commons-dbutils-1.8-javadoc.jar=115b21ecf633185aa055ce2392ce6298bc45a50278fbce10892ee9493293cdde39149f2ff739848f2557058c7e5c866dcb765e2773a28f890725ddf328c73259
> >
> >
> commons-dbutils-1.8-sources.jar.asc=2e1709b4f9dfc1f9320127dded40b9af9c19a5e249aaac9deadd38aee572a28ecb7ef8de559b93c97b6093596cadd70e180b99787b82965e0f5d8376adfac8fb
> >
> >
> commons-dbutils-1.8-bin.zip.asc=e5cb25909f68e5d0a3249ca484d10b97b859ed56e92072b13ca64c85d18686605189703e2e146343247b6a64e3e20b4c7e07e12f40b37c87d305a9d71e951581
> >
> >
> commons-dbutils-1.8-bin.tar.gz=c8a9cf0c59a64cbe4717edeb377a85fd8395c4bbd9eb124d24099d04fd6e8e3b87063bf4bfaca63e58d8deb80a651b4d9fc03c00db56f0ea0d93053f61138f4f
> >
> >
> commons-dbutils-1.8-src.tar.gz=5277ebf4ed36a4301ce7a6280885360ea51ebc82771a01405684397fa33144c00214e5ac7a3b4aad75ec4dd6d3eb46ef083321bf03414dce29982c553da1360e
> >
> >
> commons-dbutils-1.8-javadoc.jar.asc=9a6bca512db25dcd6be9215862b1f04cfd86ad58f6c21d3cbcebe1ad9137e5dc3417d05db915acc4fc6f22ef0c40893c401a77568148d714f66375ea66da2078
> >
> >
> commons-dbutils-1.8-src.tar.gz.asc=d7ace200a63c2ee705cda7d4d5b62d401b8cb9e54b5b4a582e3e8f0c62f16ac964ebf5f440ad39f60344d28771f6942833c241c9163ced70f5bb7dfd256989c5
> >
> >
> commons-dbutils-1.8-src.zip.asc=3c4e20ceb0a0c071cc425be3a4604651de47df2be7bf1985525d319fc3c8035299a75ba558172411338d0174d504cddd7c22dc35d334a21b043f885fdabcd736
> >
> >
> commons-dbutils-1.8-tests.jar.asc=a9df8e97c85f437223ae0ba02fafba542c978a44ec12a61cf742e261e238e4bd9a4d7bb1239c8101aeede7d88ba4ad647ceb2eaececb1cc3c4027d9308491a00
> >
> >
> commons-dbutils-1.8-sources.jar=425dcae024bc592ce38238fde0e2d7eaf919fa238d64e23796c383afa30892d6c4a7850e9f405fe7d0fcd32dde97568f1ef2e708cbb193311746744ce0425fa0
> >
> >
> commons-dbutils-1.8-bin.zip=ee4819761efccc4ba3de929672a81415a787775eff0175ea782687a669e305b159e483f21e38f020e5459e3aff11adff02aa762d4aa7c0b35a2925e45730d61d
> >
> >
> commons-dbutils-1.8.jar.asc=e7555a17fd6a85aa0b80d54e163809bfae839a5b23f4e57c5d804fb039ec37599df08d45dadef8b33bc7ee26d17c1612062bdaf2c07ecc576773c583ed9048ff
> >
> >
> commons-dbutils-1.8-src.zip=5363310bda5f09337992f7d2ed4b73b3d76d8107101584a1d14487a55665476ff47989fb4e2ddbbe139b5fe36faf7a7b14d99aaf89cb8dc48c605531d61d35b9
> >
> >
> > (no need for .asc hashes!)
> >
> > I have tested this with ***'mvn clean install site'*** using:
> > ***
> > Use the output from "mvn -version" for each combination you tested.
> > ***
> >
> > Details of changes since 1.7 are in the release notes:
> >
> >
> https://dist.apache.org/repos/dist/dev/commons/dbutils/1.8-RC2/RELEASE-NOTES.txt
> >
> >
> https://dist.apache.org/repos/dist/dev/commons/dbutils/1.8-RC2/site/changes-report.html
> >
> > Site:
> >
> >
> https://dist.apache.org/repos/dist/dev/commons/dbutils/1.8-RC2/site/index.html
> >     (note some *relative* links are broken and the 1.8 directories are
> not
> > yet created - these will be OK once the site is deployed.)
> >
> > CLIRR Report (compared to 1.7):
> >
> >
> https://dist.apache.org/repos/dist/dev/commons/dbutils/1.8-RC2/site/clirr-report.html
> >
> > JApiCmp Report (compared to 1.7):
> >
> >
> https://dist.apache.org/repos/dist/dev/commons/dbutils/1.8-RC2/site/japicmp.html
> >
> >     ***
> >     Note that Clirr reports several errors.
> >     These are considered OK for the reasons stated below.
> >     These exceptions are also noted in the Changes and Release Notes.
> >
> >     Errors reported:
> >     - methods added to interface: OK because that does not affect binary
> > compatibility.
> >     - etc.
> >     ***
> >
> > RAT Report:
> >
> >
> https://dist.apache.org/repos/dist/dev/commons/dbutils/1.8-RC2/site/rat-report.html
> >
> > KEYS:
> >   https://www.apache.org/dist/commons/KEYS
> >
> > Please review the release candidate and vote.
> > This vote will close no sooner that 72 hours from now.
> >
> >   [ ] +1 Release these artifacts
> >   [ ] +0 OK, but...
> >   [ ] -0 OK, but really should fix...
> >   [ ] -1 I oppose this release because...
> >
> > Thank you,
> >
> > Carl Hall,
> > Release Manager (using key 1e5ab6d3cf8ebf5f)
> >
> > For following is intended as a helper and refresher for reviewers.
> >
> > Validating a release candidate
> > ==============================
> >
> > These guidelines are NOT complete.
> >
> > Requirements: Git, Java, Maven.
> >
> > You can validate a release from a release candidate (RC) tag as follows.
> >
> > 1) Clone and checkout the RC tag
> >
> > git clone https://gitbox.apache.org/repos/asf/commons-dbutils.git
> > --branch commons-dbutils-1.8-RC2 commons-dbutils-1.8-RC2
> > cd commons-dbutils-1.8-RC2
> >
> > 2) Check Apache licenses
> >
> > This step is not required if the site includes a RAT report page which
> you
> > then must check.
> >
> > mvn apache-rat:check
> >
> > 3) Check binary compatibility
> >
> > Older components still use Apache Clirr:
> >
> > This step is not required if the site includes a Clirr report page which
> > you then must check.
> >
> > mvn clirr:check
> >
> > Newer components use JApiCmp with the japicmp Maven Profile:
> >
> > This step is not required if the site includes a JApiCmp report page
> which
> > you then must check.
> >
> > mvn install -DskipTests -P japicmp japicmp:cmp
> >
> > 4) Build the package
> >
> > mvn -V clean package
> >
> > You can record the Maven and Java version produced by -V in your VOTE
> > reply.
> > To gather OS information from a command line:
> > Windows: ver
> > Linux: uname -a
> >
> > 5) Build the site for a single module project
> >
> > Note: Some plugins require the components to be installed instead of
> > packaged.
> >
> > mvn site
> > Check the site reports in:
> > - Windows: target\site\index.html
> > - Linux: target/site/index.html
> >
> > 6) Build the site for a multi-module project
> >
> > mvn site
> > mvn site:stage
> > Check the site reports in:
> > - Windows: target\site\index.html
> > - Linux: target/site/index.html
> >
> > -the end-
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> > For additional commands, e-mail: dev-h...@commons.apache.org
> >
>

Reply via email to