Return-Path: Delivered-To: apmail-commons-dev-archive@www.apache.org Received: (qmail 26916 invoked from network); 17 Nov 2007 10:50:06 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 17 Nov 2007 10:50:06 -0000 Received: (qmail 65197 invoked by uid 500); 17 Nov 2007 10:49:52 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 65124 invoked by uid 500); 17 Nov 2007 10:49:52 -0000 Mailing-List: contact dev-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Jakarta Commons Developers List" Delivered-To: mailing list dev@commons.apache.org Received: (qmail 65115 invoked by uid 99); 17 Nov 2007 10:49:52 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 17 Nov 2007 02:49:52 -0800 X-ASF-Spam-Status: No, hits=1.2 required=10.0 tests=SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (athena.apache.org: local policy) Received: from [209.85.146.179] (HELO wa-out-1112.google.com) (209.85.146.179) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 17 Nov 2007 10:49:43 +0000 Received: by wa-out-1112.google.com with SMTP id k34so1328163wah for ; Sat, 17 Nov 2007 02:49:36 -0800 (PST) Received: by 10.114.13.1 with SMTP id 1mr1591582wam.1195296575899; Sat, 17 Nov 2007 02:49:35 -0800 (PST) Received: by 10.114.73.5 with HTTP; Sat, 17 Nov 2007 02:49:35 -0800 (PST) Message-ID: <935068bf0711170249i2d92a2dfo66d083596ab5dd13@mail.gmail.com> Date: Sat, 17 Nov 2007 21:49:35 +1100 From: "Dion Gillard" To: "Jakarta Commons Developers List" Subject: Re: svn commit: r595632 - in /commons/proper/jexl/trunk: pom.xml src/assembly/ src/assembly/bin.xml src/assembly/src.xml src/site/ src/site/resources/ src/site/resources/images/ src/site/site.xml In-Reply-To: <55afdc850711162127h92f743obe70ae17dd1fb87f@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20071116114303.9C12A1A9832@eris.apache.org> <935068bf0711160503y6de0b20bm7a55bff64576711b@mail.gmail.com> <473DD88F.2030200@mvdb.net> <935068bf0711161710r6eefe493sfbbd8602528f9cb3@mail.gmail.com> <55afdc850711162127h92f743obe70ae17dd1fb87f@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org Let me prefix this with: - I'm not against us using mvn as our build tool for components. - I think we need to do this more consensually and build more knowledge about the tools we use. On 11/17/07, Niall Pemberton wrote: > On Nov 17, 2007 1:10 AM, Dion Gillard wrote: > > I figure that things are going to get missed since there's been little > > to no discussion I can recall in recent times on how, what, when it's > > happening. > > The reality is its already been happening slowly. Ten components had > working m2 builds (and proxy and jci only have m2 builds) before I > started yesterday. Another 15 had m2 poms but needed work on the site > generation and/or needed assembly defs. adding to create distros. Only > nine components had no m2 build at all (two still don't - Attributes > and Jelly). We've also had five releases of the commons-parent pom - > the vote for each of which was an opportunity for discussion. How would devs without good working mvn knowledge be able to vote on a release of the parent pom? It's a 421 line beast that requires a very good understanding of mvn internals to provide constructive criticism, and given that under half of the components weren't using mvn to perform releases, I would expect that the vote would be of interest only to those already with a supported mvn build. The pom.xml for Jexl is 167 lines long. Add the parent pom's 421 lines and the org.apache:apache:4 pom (where would I find that in svn out of curiosity? Where are it's votes happening?), and compare it to the 198 line Jexl standalone project.xml. mvn is almost three times more complex in this case. > From my PoV I just kick started the process yesterday by ensuring that > the remaining components (expect Attributes and Jelly) now have m2 > builds that can 1) Generate the component's Site and 2) Produce source > and binary distros. Its quite possible that I made some mistakes and > missed things (I did test each one), but m2-wise I believe those > components are now in a better state and it should be easier for the > devs on each component to refine them as they require. The main point > is though that these are all in addition to the existing m1/ant builds > which should continue to work. I suppose the question is, if the devs on those components don't have experience with mvn and it's settings who will maintain these files? Will we repeat the mistake we made with some components when we added a maven build?The ant builds either got updated and out of sync with the maven build, or the ant builds got left behind and in many cases weren't very useful, even though they should have continued to work. Was there a reason to leave out Jelly and Attributes from the updates, or was it just lack of time and understanding of the existing maven build for them? > "Moving" to m2 implies giving up the m1 builds - I agree we haven't > yet taken that decision, but with the m2 coverage we've had up till > this point that wasn't possible. I think that now that almost all > components have a working m2 build its now become an option - but > you're right that sort of decision needs discussion. What about using buildr? > > > As a start: > > - Shouldn't things like the changes plugin issue management URL be in > > the parent POM? > > How do you mean? I guess we could do something like the following in > the parent pom: > > http://commons.apache.org/${commons.id}/ > > jira > http://issues.apache.org/jira/browse/${commons.id} > > > scm:svn:http://svn.apache.org/repos/asf/commons/proper/${commons.id}/trunk/ > scm:svn:https://svn.apache.org/repos/asf/commons/proper/${commons.id}/trunk/ > http://svn.apache.org/viewvc/commons/proper/${commons.id}/trunk/ > > > And then in individual component pom's: > > validator > > > Was that the kind of thing - or something else? Yep, that's it. I'd rather we didn't repeat ourselves across components without a good reason. > > - Is there a working Javacc 4.0 plugin for mvn? > > Dunno, but not sure of the relevance - assume its specific to a couple > of components. If there isn't I wont be able to move the v2 branch of Jexl to mvn. > > Niall > > > Dion > > > > > > On 11/17/07, Martin van den Bemt wrote: > > > Do you anticipate complications with a move ? > > > > > > Mvgr, > > > Martin > > > > > > Dion Gillard wrote: > > > > Hi all, > > > > > > > > is there a mandate for an m2 build for commons? > > > > > > > > On 11/16/07, niallp@apache.org wrote: > > > >> Author: niallp > > > >> Date: Fri Nov 16 03:42:56 2007 > > > >> New Revision: 595632 > > > >> > > > >> URL: http://svn.apache.org/viewvc?rev=595632&view=rev > > > >> Log: > > > >> Add m2 build > > > >> > > > >> Added: > > > >> commons/proper/jexl/trunk/pom.xml (with props) > > > >> commons/proper/jexl/trunk/src/assembly/ > > > >> commons/proper/jexl/trunk/src/assembly/bin.xml (with props) > > > >> commons/proper/jexl/trunk/src/assembly/src.xml (with props) > > > >> commons/proper/jexl/trunk/src/site/ > > > >> commons/proper/jexl/trunk/src/site/resources/ > > > >> commons/proper/jexl/trunk/src/site/resources/images/ > > > >> - copied from r595528, commons/proper/jexl/trunk/xdocs/images/ > > > >> commons/proper/jexl/trunk/src/site/site.xml (with props) > > > >> > > > >> Added: commons/proper/jexl/trunk/pom.xml > > > >> URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/pom.xml?rev=595632&view=auto > > > >> ============================================================================== > > > >> --- commons/proper/jexl/trunk/pom.xml (added) > > > >> +++ commons/proper/jexl/trunk/pom.xml Fri Nov 16 03:42:56 2007 > > > >> @@ -0,0 +1,167 @@ > > > >> + > > > >> + > > > >> + > > >> + xmlns="http://maven.apache.org/POM/4.0.0" > > > >> + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" > > > >> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd"> > > > >> + > > > >> + org.apache.commons > > > >> + commons-parent > > > >> + 5 > > > >> + > > > >> + 4.0.0 > > > >> + commons-jexl > > > >> + commons-jexl > > > >> + 1.1.1-SNAPSHOT > > > >> + Commons JEXL > > > >> + 2003 > > > >> + Jexl is an implementation of the JSTL Expression Language with extensions. > > > >> + http://commons.apache.org/jexl/ > > > >> + > > > >> + > > > >> + jira > > > >> + http://issues.apache.org/jira/browse/JEXL > > > >> + > > > >> + > > > >> + > > > >> + scm:svn:http://svn.apache.org/repos/asf/commons/proper/jexl/trunk > > > >> + scm:svn:https://svn.apache.org/repos/asf/commons/proper/jexl/trunk > > > >> + http://svn.apache.org/viewvc/commons/proper/jexl/trunk > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + dIon Gillard > > > >> + dion > > > >> + dion@apache.org > > > >> + Apache Software Foundation > > > >> + > > > >> + > > > >> + Geir Magnusson Jr. > > > >> + geirm > > > >> + geirm@apache.org > > > >> + independent > > > >> + > > > >> + > > > >> + Tim O'Brien > > > >> + tobrien > > > >> + tobrien@apache.org > > > >> + independent > > > >> + > > > >> + > > > >> + Peter Royal > > > >> + proyal > > > >> + proyal@apache.org > > > >> + Apache Software Foundation > > > >> + > > > >> + > > > >> + James Strachan > > > >> + jstrachan > > > >> + jstrachan@apache.org > > > >> + SpiritSoft, Inc. > > > >> + > > > >> + > > > >> + Rahul Akolkar > > > >> + rahul > > > >> + rahul AT apache.org > > > >> + Apache Software Foundation > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + commons-logging > > > >> + commons-logging > > > >> + 1.0.4 > > > >> + > > > >> + > > > >> + junit > > > >> + junit > > > >> + 3.8.1 > > > >> + true > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + src/java > > > >> + src/test > > > >> + > > > >> + > > > >> + . > > > >> + META-INF > > > >> + > > > >> + NOTICE.txt > > > >> + LICENSE.txt > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + org.apache.maven.plugins > > > >> + maven-surefire-plugin > > > >> + > > > >> + > > > >> + **/Test*.java > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + maven-assembly-plugin > > > >> + > > > >> + > > > >> + src/assembly/bin.xml > > > >> + src/assembly/src.xml > > > >> + > > > >> + gnu > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + org.apache.maven.plugins > > > >> + maven-changes-plugin > > > >> + > > > >> + ${basedir}/xdocs/changes.xml > > > >> + %URL%/%ISSUE% > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + changes-report > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> > > > >> Propchange: commons/proper/jexl/trunk/pom.xml > > > >> ------------------------------------------------------------------------------ > > > >> svn:eol-style = native > > > >> > > > >> Propchange: commons/proper/jexl/trunk/pom.xml > > > >> ------------------------------------------------------------------------------ > > > >> svn:keywords = Date Author Id Revision HeadURL > > > >> > > > >> Added: commons/proper/jexl/trunk/src/assembly/bin.xml > > > >> URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/assembly/bin.xml?rev=595632&view=auto > > > >> ============================================================================== > > > >> --- commons/proper/jexl/trunk/src/assembly/bin.xml (added) > > > >> +++ commons/proper/jexl/trunk/src/assembly/bin.xml Fri Nov 16 03:42:56 2007 > > > >> @@ -0,0 +1,44 @@ > > > >> + > > > >> + > > > >> + bin > > > >> + > > > >> + tar.gz > > > >> + zip > > > >> + > > > >> + false > > > >> + > > > >> + > > > >> + > > > >> + LICENSE.txt > > > >> + NOTICE.txt > > > >> + RELEASE-NOTES.txt > > > >> + > > > >> + > > > >> + > > > >> + target > > > >> + > > > >> + > > > >> + *.jar > > > >> + > > > >> + > > > >> + > > > >> + target/site/apidocs > > > >> + apidocs > > > >> + > > > >> + > > > >> + > > > >> > > > >> Propchange: commons/proper/jexl/trunk/src/assembly/bin.xml > > > >> ------------------------------------------------------------------------------ > > > >> svn:eol-style = native > > > >> > > > >> Propchange: commons/proper/jexl/trunk/src/assembly/bin.xml > > > >> ------------------------------------------------------------------------------ > > > >> svn:keywords = Date Author Id Revision HeadURL > > > >> > > > >> Added: commons/proper/jexl/trunk/src/assembly/src.xml > > > >> URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/assembly/src.xml?rev=595632&view=auto > > > >> ============================================================================== > > > >> --- commons/proper/jexl/trunk/src/assembly/src.xml (added) > > > >> +++ commons/proper/jexl/trunk/src/assembly/src.xml Fri Nov 16 03:42:56 2007 > > > >> @@ -0,0 +1,50 @@ > > > >> + > > > >> + > > > >> + src > > > >> + > > > >> + tar.gz > > > >> + zip > > > >> + > > > >> + ${artifactId}-${version}-src > > > >> + > > > >> + > > > >> + > > > >> + build.properties.sample > > > >> + build.xml > > > >> + build-maven.xml > > > >> + LICENSE.txt > > > >> + maven.xml > > > >> + NOTICE.txt > > > >> + pom.xml > > > >> + project.properties > > > >> + project.xml > > > >> + PROPOSAL.html > > > >> + RELEASE-NOTES.txt > > > >> + > > > >> + > > > >> + > > > >> + examples > > > >> + > > > >> + > > > >> + src > > > >> + > > > >> + > > > >> + xdocs > > > >> + > > > >> + > > > >> + > > > >> > > > >> Propchange: commons/proper/jexl/trunk/src/assembly/src.xml > > > >> ------------------------------------------------------------------------------ > > > >> svn:eol-style = native > > > >> > > > >> Propchange: commons/proper/jexl/trunk/src/assembly/src.xml > > > >> ------------------------------------------------------------------------------ > > > >> svn:keywords = Date Author Id Revision HeadURL > > > >> > > > >> Added: commons/proper/jexl/trunk/src/site/site.xml > > > >> URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/site/site.xml?rev=595632&view=auto > > > >> ============================================================================== > > > >> --- commons/proper/jexl/trunk/src/site/site.xml (added) > > > >> +++ commons/proper/jexl/trunk/src/site/site.xml Fri Nov 16 03:42:56 2007 > > > >> @@ -0,0 +1,47 @@ > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + Commons JEXL > > > >> + /images/jexl-logo-white.png > > > >> + /index.html > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> + > > > >> > > > >> Propchange: commons/proper/jexl/trunk/src/site/site.xml > > > >> ------------------------------------------------------------------------------ > > > >> svn:eol-style = native > > > >> > > > >> Propchange: commons/proper/jexl/trunk/src/site/site.xml > > > >> ------------------------------------------------------------------------------ > > > >> svn:keywords = Date Author Id Revision HeadURL > > > >> > > > >> > > > >> > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > > > For additional commands, e-mail: dev-help@commons.apache.org > > > > > > > > > > > > -- > > dIon Gillard > > Rule #131 of Acquisition: Information is Profit. > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > > For additional commands, e-mail: dev-help@commons.apache.org > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > For additional commands, e-mail: dev-help@commons.apache.org > > -- dIon Gillard Rule #131 of Acquisition: Information is Profit. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org For additional commands, e-mail: dev-help@commons.apache.org