We agreed that each new recut is a new point release.
Cheers, Paul On Sat, Dec 13, 2014 at 3:42 PM, Igor Fedorenko <i...@ifedorenko.com> wrote: > > Why? How will we tell the original broken binaries from the new ones? > > On December 13, 2014 4:01:31 PM EST, Jason van Zyl <ja...@takari.io> > wrote: > >No, it will be 3.2.4. > > > >On Dec 13, 2014, at 3:52 PM, Stephen Connolly > ><stephen.alan.conno...@gmail.com> wrote: > > > >> Iirc we said not reusing version numbers after a .0 so this will be > >3.2.5, > >> yes? > >> > >> On Saturday, December 13, 2014, Jason van Zyl <ja...@takari.io> > >wrote: > >> > >>> The fixes have been made, I'll recut the release. > >>> > >>> On Dec 13, 2014, at 9:44 AM, Igor Fedorenko <i...@ifedorenko.com > >>> <javascript:;>> wrote: > >>> > >>>> I've already reintroduced DefaultJavaToolChain and Tycho is happy > >now > >>> [1]. > >>>> > >>>> Tycho needs access to DefaultJavaToolChain#getJavaHome() which, to > >the > >>>> best of my knowledge, is not available from any other API, is not > >>>> available through ToolchainManager. > >>>> > >>>> I don't believe Tycho references JavaToolChain interface. As a side > >>>> note, in the future we should avoid case-only renames like > >>>> JavaToolChain->JavaToolchain, these cause problems on > >case-insensitive > >>>> filesystems, like the ones used by Windows and OSX. > >>>> > >>>> We don't distinguish between public and internal classes in Maven, > >at > >>>> least not generally. There are few classes that are explicitly > >marked as > >>>> @provisional, like ArtifactDescriptorReaderDelegate for example, > >but > >>>> this is not enforced and majority of classes are not marked in any > >way. > >>>> I agree we need to have a mechanism to distinguish between public > >and > >>>> internal classes and I have few ideas to do this, but I don't think > >we > >>>> can do this retroactively. We have to keep most/all existing > >classes and > >>>> treat them as public API, unfortunately. Maybe mark them as > >deprecated > >>>> when we know we will likely change them in the future, but any real > >>>> change has to wait Maven 4, and I am not sure we'll be able to > >afford > >>>> massive breakage even then. > >>>> > >>>> [1] > >>> > > > https://git-wip-us.apache.org/repos/asf?p=maven.git;a=commit;h=d6e45a5d56ab0facd36751ccee722db6a2006f50 > >>>> > >>>> -- > >>>> Regards, > >>>> Igor > >>>> > >>>> On 2014-12-13, 9:04, Hervé BOUTEMY wrote: > >>>>> ok, I had a look at Tycho sources: > >>>>> this is something introduced recently (10/10/2014): IIUC, Tycho > >0.22.0 > >>> was > >>>>> released since then > >>>>> > >>>>> I'm surprised of tycho-core's ToolchainProvider: IIUC, that's a > >rewrite > >>> of a > >>>>> part of maven-toochains-plugin, depending on > >ToolchainManagerPrivate > >>> which is > >>>>> not part of public API > >>>>> IIUC, the intent was to get the configured toolchain: since then, > >I > >>> wrote a > >>>>> little doc for that [1] to show how to use ToolchainManager to do > >>> exactly that > >>>>> using normal Maven API > >>>>> > >>>>> Notice this won't fix everything, since: > >>>>> 1. Tycho needs JavaToochain, which has been renamed (Toolchain vs > >>> ToolChain) > >>>>> 2. the interface doesn't offer expected getJavaHome() API > >>>>> > >>>>> > >>>>> Then you'll need to cast to DefaultJavaToolChain, which will need > >to be > >>> added > >>>>> back to maven-core: I'll do it immediately. > >>>>> > >>>>> I don't know the Tycho community, but using ToolchainManager would > >IMHO > >>> be > >>>>> more future-proof than writing ToolchainProvider > >>>>> > >>>>> Regards, > >>>>> > >>>>> Hervé > >>>>> > >>>>> > >>>>> [1] > >>> > > > http://maven.apache.org/plugins/maven-toolchains-plugin/toolchains/custom.html > >>>>> > >>>>> > >>>>> Le samedi 13 décembre 2014 09:13:50 Hervé BOUTEMY a écrit : > >>>>>> DefaultJavaToolChain? you mean the implementation? > >>>>>> > >>>>>> Can you give me pointers to the Tycho sources that use this API? > >>>>>> (that's clearly not expected) > >>>>>> > >>>>>> I'm in favor of introducing deprecated DefaultJavaToolChain > >>>>>> that extends the new implementation, which is easy to do: just > >need to > >>>>>> understand how it is used in Tycho, since the class is supposed > >to be > >>> used > >>>>>> by JavaToolchainFactory (on only this one). > >>>>>> > >>>>>> Regards, > >>>>>> > >>>>>> Hervé > >>>>>> > >>>>>> Le vendredi 12 décembre 2014 18:38:44 Igor Fedorenko a écrit : > >>>>>>> Unfortunately, I have to take this back. The changes to > >toolchain > >>>>>>> broke Tycho and, short of using reflection, I don't see how > >>>>>>> to make Tycho work with maven 3.2.4 and earlier versions of > >Maven. > >>>>>>> > >>>>>>> For better or worse, > >>>>>>> org.apache.maven.toolchain.java.DefaultJavaToolChain was part of > >>>>>>> Maven API since 2.x and I don't think we can just remove the > >class. > >>>>>>> > >>>>>>> I see two ways to fix this. Either we rename the classes back, > >which > >>> is > >>>>>>> probably the easiest. Or we introduce deprecated > >DefaultJavaToolChain > >>>>>>> that extends the new implementation. > >>>>>>> > >>>>>>> Here is my -1 for releasing 3.2.4 in its current state. > >>>>>>> > >>>>>>> -- > >>>>>>> Regards, > >>>>>>> Igor > >>>>>>> > >>>>>>> On 2014-12-12, 18:01, Igor Fedorenko wrote: > >>>>>>>> +1 > >>>>>>>> > >>>>>>>> -- > >>>>>>>> Regards, > >>>>>>>> Igor > >>>>>>>> > >>>>>>>> On 2014-12-12, 16:54, Jason van Zyl wrote: > >>>>>>>>> Hi, > >>>>>>>>> > >>>>>>>>> Time to release Maven 3.2.4! > >>>>>>>>> > >>>>>>>>> Here is a link to Jira with 20 issues resolved: > >>>>>>>>> > >>> > >https://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=10500&versi > >>>>>>>>> on > >>>>>>>>> =20574 > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> Staging repo: > >>>>>>>>> https://repository.apache.org/content/repositories/maven-1102/ > >>>>>>>>> > >>>>>>>>> The distributable binaries and sources for testing can be > >found > >>> here: > >>>>>>>>> > >>> > >https://repository.apache.org/content/repositories/maven-1102/org/apach > >>>>>>>>> e/ > >>>>>>>>> maven/apache-maven/3.2.4/ > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> Specifically the zip, tarball, and source archives can be > >found > >>> here: > >>>>>>>>> > >>> > >https://repository.apache.org/content/repositories/maven-1102/org/apach > >>>>>>>>> e/ > >>>>>>>>> maven/apache-maven/3.2.4/apache-maven-3.2.4-bin.zip > >>>>>>>>> > >>>>>>>>> > >>> > >https://repository.apache.org/content/repositories/maven-1102/org/apach > >>>>>>>>> e/ > >>>>>>>>> maven/apache-maven/3.2.4/apache-maven-3.2.4-bin.tar.gz > >>>>>>>>> > >>>>>>>>> > >>> > >https://repository.apache.org/content/repositories/maven-1102/org/apach > >>>>>>>>> e/ > >>>>>>>>> maven/apache-maven/3.2.4/apache-maven-3.2.4-src.zip > >>>>>>>>> > >>>>>>>>> > >>> > >https://repository.apache.org/content/repositories/maven-1102/org/apach > >>>>>>>>> e/ > >>>>>>>>> maven/apache-maven/3.2.4/apache-maven-3.2.4-src.tar.gz > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> Source release checksum(s): > >>>>>>>>> apache-maven-3.2.4-src.zip sha1: > >>>>>>>>> db5fb9feda693f05fe7effaf096d6cd4dd5eff44 > >>>>>>>>> > >>>>>>>>> Staging site: > >>>>>>>>> http://takari.io/maven-3.2.4/ > >>>>>>>>> > >>>>>>>>> Vote open for 72 hours. > >>>>>>>>> > >>>>>>>>> [ ] +1 > >>>>>>>>> [ ] +0 > >>>>>>>>> [ ] -1 > >>>>>>>>> > >>>>>>>>> Thanks, > >>>>>>>>> > >>>>>>>>> The Maven Team > >>>>>>>>> > >>> > >--------------------------------------------------------------------- > >>>>>>>>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > >>> <javascript:;> > >>>>>>>>> For additional commands, e-mail: dev-h...@maven.apache.org > >>> <javascript:;> > >>>>>>>> > >>>>>>>> > >--------------------------------------------------------------------- > >>>>>>>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > >>> <javascript:;> > >>>>>>>> For additional commands, e-mail: dev-h...@maven.apache.org > >>> <javascript:;> > >>>>>>> > >>>>>>> > >--------------------------------------------------------------------- > >>>>>>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > >>> <javascript:;> > >>>>>>> For additional commands, e-mail: dev-h...@maven.apache.org > >>> <javascript:;> > >>>>>> > >>>>>> > >--------------------------------------------------------------------- > >>>>>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > >>> <javascript:;> > >>>>>> For additional commands, e-mail: dev-h...@maven.apache.org > >>> <javascript:;> > >>>>> > >>>>> > >>>>> > >--------------------------------------------------------------------- > >>>>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > ><javascript:;> > >>>>> For additional commands, e-mail: dev-h...@maven.apache.org > >>> <javascript:;> > >>>>> > >>>> > >>>> > >--------------------------------------------------------------------- > >>>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > ><javascript:;> > >>>> For additional commands, e-mail: dev-h...@maven.apache.org > >>> <javascript:;> > >>>> > >>> > >>> Thanks, > >>> > >>> Jason > >>> > >>> ---------------------------------------------------------- > >>> Jason van Zyl > >>> Founder, Apache Maven > >>> http://twitter.com/jvanzyl > >>> http://twitter.com/takari_io > >>> --------------------------------------------------------- > >>> > >>> The modern conservative is engaged in one of man's oldest exercises > >in > >>> moral philosophy; that is, > >>> the search for a superior moral justification for selfishness. > >>> > >>> -- John Kenneth Galbraith > >>> > >>> > >>> > >>> > >>> > >>> > >>> > >>> > >>> > >>> > >> > >> -- > >> Sent from my phone > > > >Thanks, > > > >Jason > > > >---------------------------------------------------------- > >Jason van Zyl > >Founder, Apache Maven > >http://twitter.com/jvanzyl > >http://twitter.com/takari_io > >--------------------------------------------------------- > > > >To think is easy. To act is hard. But the hardest thing in the world is > >to act in accordance with your thinking. > > > > -- Johann von Goethe > > -- > Sent from my Android device with K-9 Mail. Please excuse my brevity. > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > For additional commands, e-mail: dev-h...@maven.apache.org > >