Return-Path: Delivered-To: apmail-geronimo-dev-archive@www.apache.org Received: (qmail 22893 invoked from network); 8 Oct 2008 20:47:51 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 8 Oct 2008 20:47:51 -0000 Received: (qmail 12188 invoked by uid 500); 8 Oct 2008 20:47:48 -0000 Delivered-To: apmail-geronimo-dev-archive@geronimo.apache.org Received: (qmail 12136 invoked by uid 500); 8 Oct 2008 20:47:48 -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 12125 invoked by uid 99); 8 Oct 2008 20:47:48 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 08 Oct 2008 13:47:48 -0700 X-ASF-Spam-Status: No, hits=0.1 required=10.0 tests=DNS_FROM_SECURITYSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of linsun.unc@gmail.com designates 209.85.217.21 as permitted sender) Received: from [209.85.217.21] (HELO mail-gx0-f21.google.com) (209.85.217.21) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 08 Oct 2008 20:46:46 +0000 Received: by gxk14 with SMTP id 14so8602970gxk.19 for ; Wed, 08 Oct 2008 13:46:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to :subject:in-reply-to:mime-version:content-type :content-transfer-encoding:content-disposition:references; bh=unTlvSFVCZlcUaxMzQca1KZ3+9sUYUhqO6G/oFj/gnE=; b=OZ1WP7P7ThrgAepGKQ6ghkPEDvC5Qnhle8+gpPjNT/HRmgCfaSJLOph5PCOlWN1DYF kmngpJ/lghxU3ZziQMO0ReNqVwNlVfoDDlOP1+L18NAxLaM40vwwQGssFAAei6cMfoUT WD5BKFbtFKeyUitoTButJiGav76FNuT4SrqKM= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=Z8XysLGqncOiTlfxw5UtCWCk/7KTakpv4gNbhhFP4aFSCSsuH62BfRxpXOsfloIT9j DXr9jxv5A25JteJklitceBYJstUqcvBJfEApGdac7yEGtMLb2+H2LG9s6KsRgkqQ4hJx BRbGqDofWHW1wBq+Hgw2NWRu/raIFh5J67s68= Received: by 10.151.12.4 with SMTP id p4mr11938044ybi.183.1223498782246; Wed, 08 Oct 2008 13:46:22 -0700 (PDT) Received: by 10.150.137.19 with HTTP; Wed, 8 Oct 2008 13:46:21 -0700 (PDT) Message-ID: Date: Wed, 8 Oct 2008 16:46:21 -0400 From: "Lin Sun" To: dev@geronimo.apache.org Subject: Re: svn commit: r702586 - in /geronimo/server/trunk/plugingroups/console-jetty: ./ pom.xml src/ src/main/ src/main/history/ src/main/history/dependencies.xml src/main/plan/ In-Reply-To: <48ED13BE.8060602@earthlink.net> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20081007191000.9F73A23888AF@eris.apache.org> <48EC22F4.6030509@apache.org> <48ED13BE.8060602@earthlink.net> X-Virus-Checked: Checked by ClamAV on apache.org Hi Joe, I am having trouble to understand the difference between what you propose & what has already been done. Basically, IIUC, you propose to use a plugin for plugin group, with the attribute of pluginGroup=true to indicate that is a plugin group. This is exactly what has been done today, as we don't differenriate plugin group any other way other than the attribute of pluginGroup. If I misunderstand you, could you please give a concrete example? P.S. I think you were shot down by David because of the complexity of having plugin groups listed as dependencies on the geronimo specific plans. Lin On Wed, Oct 8, 2008 at 4:10 PM, Joe Bohn wrote: > I agree that groups of plugins are useful and perhaps necessary from a user > perspective to help eliminate the clutter. However, there are several ways > to provide for those groups. > > The way that has thus far been pursued has been creating a new module type > (is that what you would call it?) of plugingroup. > > I had proposed earlier that we just use plugins for this purpose. We can > create plugins that do nothing more than aggregate other more granular > plugins. We could still keep the attribute of plugin-group around as a > qualifier to filter out these special "aggregate plugins" from among all of > the other plugins when necessary (such as when assembling a server or to > present the user with a non-expert view of plugin management). When I > proposed this earlier I was shot down because of the classloader creation. > However, since David included a change to omit the classloader if there is > no plan ... then perhaps there is no real inhibitor to this approach now > since a plan is not needed for an aggregate plugin. > > I originally favored this approach because I felt it was the most intuitive > approach, ensured that the groupings of plugins could participate in any > scenario that a plugin can participate in today, and had the least > code/maintenance impacts. I think those benefits still hold with the > possible exception of the classloader change and what that may mean when an > aggregate plugin is used as a dependency which might require a little more > effort to resolve. > > Joe