Return-Path: Delivered-To: apmail-geronimo-dev-archive@www.apache.org Received: (qmail 13564 invoked from network); 3 Jul 2006 09:29:25 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 3 Jul 2006 09:29:25 -0000 Received: (qmail 54994 invoked by uid 500); 3 Jul 2006 09:29:23 -0000 Delivered-To: apmail-geronimo-dev-archive@geronimo.apache.org Received: (qmail 54390 invoked by uid 500); 3 Jul 2006 09:29:21 -0000 Mailing-List: contact dev-help@geronimo.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: Reply-To: dev@geronimo.apache.org List-Id: Delivered-To: mailing list dev@geronimo.apache.org Received: (qmail 54374 invoked by uid 99); 3 Jul 2006 09:29:21 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 03 Jul 2006 02:29:21 -0700 X-ASF-Spam-Status: No, hits=0.5 required=10.0 tests=DNS_FROM_RFC_ABUSE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: domain of jason.dillon@gmail.com designates 64.233.184.224 as permitted sender) Received: from [64.233.184.224] (HELO wr-out-0506.google.com) (64.233.184.224) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 03 Jul 2006 02:29:20 -0700 Received: by wr-out-0506.google.com with SMTP id i12so847051wra for ; Mon, 03 Jul 2006 02:28:59 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:mime-version:in-reply-to:references:content-type:message-id:content-transfer-encoding:subject:date:to:x-mailer:from:sender; b=e0qV96uU+MiiFBuSu/ygAXlQrzlI+AhDknfLjec67tWh6ggeW3ZoanyXR10vX/dIpPqXgjB3fBhebg2c+M14YhwNmvSfa65tXXixWhBsJ1aPMipeEAlKfz+jfx9GMe+tzcs4Nx1LXqxn2oKyQitf1eplMccJPAdla6nz0ePHsWY= Received: by 10.54.84.1 with SMTP id h1mr336313wrb; Mon, 03 Jul 2006 02:28:59 -0700 (PDT) Received: from ?10.0.1.5? ( [67.180.143.96]) by mx.gmail.com with ESMTP id g7sm3946109wra.2006.07.03.02.28.56; Mon, 03 Jul 2006 02:28:58 -0700 (PDT) Mime-Version: 1.0 (Apple Message framework v752.2) In-Reply-To: <28888961.1151917290785.JavaMail.jira@brutus> References: <28888961.1151917290785.JavaMail.jira@brutus> Content-Type: text/plain; charset=US-ASCII; delsp=yes; format=flowed Message-Id: <361D2E33-8373-4B90-8037-2C8D47E3BBD3@planet57.com> Content-Transfer-Encoding: 7bit Subject: Re: [jira] Updated: (GERONIMO-2161) [RTC] Remove Geronimo modules from dependencyManagement in root pom.xml Date: Mon, 3 Jul 2006 02:28:45 -0700 To: dev@geronimo.apache.org X-Mailer: Apple Mail (2.752.2) From: Jason Dillon Sender: Jason Dillon X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N So far 2+ days, several patches... one PMC +1, one non-PMC +1 (with caveat to ping JVZ)... now crazy problems with diff/patch.. which I'm not exactly sure how that affects the current votes... or does adding a new version of the patch negate anything else voted upon. All for work that took about an hour. I've spent much more time trying to get folks to look at it and then hack up scripts to make the build work most of the time. I don't know how much time other folks have put in... but I'm guessing that collectively we have *wasted* many hours on this one single minor change RTC. Folks, development like this will not scale... it does not scale! I'm trying to play along... but really if it is going to take this much effort for relatively minor changes that are aimed at helping to fix issues we have and move forward with projects that have been lagging for months (the m2 migration in this case) then I'm not sure how we will ever get anything done. I don't think we will attract many new committers into this type of environment. Its already resulted in several folks who had been active before switching focus to other tasks/projects. I hope they come back at some point, but I can see why they might want to apply their time in more rewarding ways. --jason On Jul 3, 2006, at 2:01 AM, Jason Dillon (JIRA) wrote: > [ http://issues.apache.org/jira/browse/GERONIMO-2161?page=all ] > > Jason Dillon updated GERONIMO-2161: > ----------------------------------- > > Attachment: GERONIMO-2161-v3.patch > > GERONIMO-2161-v3.patch is the same as v2 minus the changes to the > packaging plugin. This applied cleanly (spat out nothing with the - > s flag) on a fresh checkout of trunk with: > > {noformat} > patch -p0 -s < GERONIMO-2161-v3.patch > {noformat} > > The packaging changes are not directly related to the changes that > this issue is tracking, its additional clean up work... as well as > build debugging bits to add better logging. > > I believe that the icky script foo above should produce the same > results (sans the logging output) as the v2 patch. > > *NOTE:* I do not believe that it is needed to reapply v3 if you > already believe that v2 is acceptable. > >> [RTC] Remove Geronimo modules from dependencyManagement in root >> pom.xml >> --------------------------------------------------------------------- >> -- >> >> Key: GERONIMO-2161 >> URL: http://issues.apache.org/jira/browse/GERONIMO-2161 >> Project: Geronimo >> Type: Task >> Security: public(Regular issues) >> Components: buildsystem >> Reporter: Jason Dillon >> Assignee: Jason Dillon >> Fix For: 1.2 >> Attachments: GERONIMO-2161-configs-v1.1.sub.patch, GERONIMO-2161- >> v1.patch, GERONIMO-2161-v2.patch, GERONIMO-2161-v3.patch >> >> As I have mentioned before, I believe we should remove the >> Geronimo modules that are currently listed in the root pom.xml: >> This reduces the configuration of the pom by *~500 lines*. >> Modules that reference these as dependencies will need their pom's >> adjusted to include ${pom.version} and >> car for the configs. But in many places version >> already exists with the ${geronimoVersion} property... which kinda >> negates the purpose of the dependencyManagement section anyways. >> I believe that it is more work to keep track of every module in >> the root pom than it is to specify the additonal elements (mostly >> just ${pom.version}) in child poms. There is >> no additional maintenance, as the new elements never need to be >> changed. >> Net effect if this change is less configuration to maintain and >> thus a less brittle build that can adapt to change easier. >> Specifically these should be removed: >> {code:xml} >> >> org.apache.geronimo.modules >> ge-activemq-rar >> ${geronimoVersion} >> rar >> >> >> org.apache.geronimo.modules >> geronimo-activation >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-client >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-client-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-common >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-connector >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-connector-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-converter >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-core >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-deploy-config >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-deploy-jsr88 >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-deploy-tool >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-deployment >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-derby >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-directory >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-javamail-transport >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-j2ee >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-j2ee-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-j2ee-schema >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-kernel >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-jetty >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-jetty-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-jmx-remoting >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-mail >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-management >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-naming >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-naming-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-security >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-security-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-service-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-system >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-test-ddbean >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-timer >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-tomcat >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-tomcat-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-transaction >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-upgrade >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-util >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-web-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-webservices >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-axis >> ${geronimoVersion} >> >> >> org.apache.geronimo.modules >> geronimo-axis-builder >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> activemq >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> activemq-broker >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> axis >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> axis-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> client >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> client-corba >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> client-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> client-security >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> client-system >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> console-jetty >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> console-tomcat >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> daytrader-jetty >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> daytrader-tomcat >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> directory >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> geronimo-gbean-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> hot-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> j2ee-corba >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> j2ee-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> j2ee-security >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> j2ee-server >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> j2ee-system >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> javamail >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> jetty >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> jetty-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> jsp-examples-jetty >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> jsp-examples-tomcat >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> ldap-demo-jetty >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> ldap-demo-tomcat >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> ldap-realm >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> online-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> openejb >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> openejb-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> project.properties >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> remote-deploy-jetty >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> remote-deploy-tomcat >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> rmi-naming >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> servlets-examples-jetty >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> servlets-examples-tomcat >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> sharedlib >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> shutdown >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> system-database >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> tomcat >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> tomcat-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> uddi-jetty >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> uddi-tomcat >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> unavailable-client-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> unavailable-ejb-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> unavailable-webservices-deployer >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> upgrade >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> upgrade-cli >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> welcome-jetty >> car >> ${geronimoVersion} >> >> >> org.apache.geronimo.configs >> welcome-tomcat >> car >> ${geronimoVersion} >> >> {code} > > -- > This message is automatically generated by JIRA. > - > If you think it was sent incorrectly contact one of the > administrators: > http://issues.apache.org/jira/secure/Administrators.jspa > - > For more information on JIRA, see: > http://www.atlassian.com/software/jira >