Igor found an issue, and added an IT test to catch the issue.

I fixed the issue with a hack and added an IT test to fail for the hack...

Then I superceded the hack with a fuller fix:
https://github.com/apache/maven/commit/bd87258629db8e3fcc7aa04777afc16314c3cde0

Long term I am not 100% happy with the addition of maven-artifact as a
dependency to maven-model-builder, but until the TODOs that provide the
validation in the workspace resolver are addressed[1], I do not see any
other way out.

IMHO 3.3.5 is not safe to release, so I am changing my vote on this release
to

-1 (binding, non-veto)

With the changes in
https://github.com/apache/maven/commit/bd87258629db8e3fcc7aa04777afc16314c3cde0
I believe 3.3.6 will be our best Maven yet... modulo Jörg's memory issues,
but they are present since the 3.2.x series so I do not view those as
blocking a 3.3.6

HTH

-Stephen

[1]:
https://github.com/apache/maven/blob/bd87258629db8e3fcc7aa04777afc16314c3cde0/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java#L890

On 26 July 2015 at 12:44, Jason van Zyl <ja...@takari.io> wrote:

> So where did we get with this? Igor reported an issue but I had spotty
> internet where I was and I can’t see the message. Should I cancel this
> release? Stephen is there something you need to fix or is it done, and then
> shall I re-roll?
>
> > On Jul 20, 2015, at 3:25 PM, Jason van Zyl <ja...@takari.io> wrote:
> >
> > +1
> >
> > Analyzing release validity...
> >
> > stagingUrl:
> https://repository.apache.org/content/repositories/maven-1200
> > groupId: org.apache.maven
> > artifactId: apache-maven
> > version: 3.3.5
> >
> > Source ZIP url exists.
> >
> https://repository.apache.org/content/repositories/maven-1200/org/apache/maven/apache-maven/3.3.5/apache-maven-3.3.5-src.zip
> >
> > Source ZIP SHA1 url exists.
> >
> https://repository.apache.org/content/repositories/maven-1200/org/apache/maven/apache-maven/3.3.5/apache-maven-3.3.5-src.zip.sha1
> >
> > Binary ZIP url exists.
> >
> https://repository.apache.org/content/repositories/maven-1200/org/apache/maven/apache-maven/3.3.5/apache-maven-3.3.5-bin.zip
> >
> > Binary ZIP SHA1 url exists.
> >
> https://repository.apache.org/content/repositories/maven-1200/org/apache/maven/apache-maven/3.3.5/apache-maven-3.3.5-bin.zip.sha1
> >
> > Calculated SHA1 of source ZIP matches published SHA1 of source ZIP.
> > f2b28783fbf9a4fda47d2924a29aa99b9dc7b898
> >
> > Calculated SHA1 of binary ZIP matches published SHA1 of binary ZIP.
> > f3c1488a47701f7542aafaac8c9ddec2c91af8a6
> >
> > LICENSE file is present in the source distribution.
> > NOTICE file is present in the source distribution.
> >
> > LICENSE file is present in the binary distribution.
> > NOTICE file is present in the binary distribution.
> >
> > Git revision of release as determined from
> maven-core-3.3.5.jar:org/apache/maven/messages/build.properties(buildNumber):
> > 2226900a49396ae834b749ff22c126aed89dbf5a
> >
> > All files in source distribution have been found in the release revision
> (taking into account acceptable exclusions).
> >
> >> On Jul 20, 2015, at 3:23 PM, Jason van Zyl <ja...@takari.io> wrote:
> >>
> >> Hi,
> >>
> >> Time to release Maven 3.3.5!
> >>
> >> Here is a link to the issues resolved:
> >>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12316922&version=12333058
> >>
> >> Staging repo:
> >> https://repository.apache.org/content/repositories/maven-1200/
> >>
> >> The distributable binaries and sources for testing can be found here:
> >>
> https://repository.apache.org/content/repositories/maven-1200/org/apache/maven/apache-maven/3.3.5/
> >>
> >> Specifically the zip, tarball, and source archives can be found here:
> >>
> https://repository.apache.org/content/repositories/maven-1200/org/apache/maven/apache-maven/3.3.5/apache-maven-3.3.5-bin.zip
> >>
> https://repository.apache.org/content/repositories/maven-1200/org/apache/maven/apache-maven/3.3.5/apache-maven-3.3.5-bin.tar.gz
> >>
> https://repository.apache.org/content/repositories/maven-1200/org/apache/maven/apache-maven/3.3.5/apache-maven-3.3.5-src.zip
> >>
> https://repository.apache.org/content/repositories/maven-1200/org/apache/maven/apache-maven/3.3.5/apache-maven-3.3.5-src.tar.gz
> >>
> >> Source release checksum(s):
> >> apache-maven-3.3.5-src.zip sha1:
> f2b28783fbf9a4fda47d2924a29aa99b9dc7b898
> >>
> >> Staging site:
> >> http://people.apache.org/~jvanzyl/maven-3.3.5/
> >>
> >> Vote open for 72 hours.
> >>
> >> [ ] +1
> >> [ ] +0
> >> [ ] -1
> >>
> >> Thanks,
> >>
> >> The Maven Team
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> >> For additional commands, e-mail: dev-h...@maven.apache.org
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> >> For additional commands, e-mail: dev-h...@maven.apache.org
> >>
> >
> > Thanks,
> >
> > Jason
> >
> > ----------------------------------------------------------
> > Jason van Zyl
> > Founder, Takari and Apache Maven
> > http://twitter.com/jvanzyl
> > http://twitter.com/takari_io
> > ---------------------------------------------------------
> >
> > First, the taking in of scattered particulars under one Idea,
> > so that everyone understands what is being talked about ... Second,
> > the separation of the Idea into parts, by dividing it at the joints,
> > as nature directs, not breaking any limb in half as a bad carver might.
> >
> >  -- Plato, Phaedrus (Notes on the Synthesis of Form by C. Alexander)
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
>
> Thanks,
>
> Jason
>
> ----------------------------------------------------------
> Jason van Zyl
> Founder, Takari and Apache Maven
> http://twitter.com/jvanzyl
> http://twitter.com/takari_io
> ---------------------------------------------------------
>
> Simplex sigillum veri. (Simplicity is the seal of truth.)
>
>
>
>
>
>
>
>
>
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>

Reply via email to