Return-Path: 
 <dev-return-36370-apmail-struts-dev-archive=struts.apache.org@struts.apache.org>
Delivered-To: apmail-struts-dev-archive@www.apache.org
Received: (qmail 45204 invoked from network); 12 Aug 2005 03:06:24 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199)
  by minotaur.apache.org with SMTP; 12 Aug 2005 03:06:24 -0000
Received: (qmail 602 invoked by uid 500); 12 Aug 2005 03:06:12 -0000
Delivered-To: apmail-struts-dev-archive@struts.apache.org
Received: (qmail 585 invoked by uid 500); 12 Aug 2005 03:06:12 -0000
Mailing-List: contact dev-help@struts.apache.org; run by ezmlm
Precedence: bulk
List-Unsubscribe: <mailto:dev-unsubscribe@struts.apache.org>
List-Help: <mailto:dev-help@struts.apache.org>
List-Post: <mailto:dev@struts.apache.org>
List-Id: "Struts Developers List" <dev.struts.apache.org>
Reply-To: "Struts Developers List" <dev@struts.apache.org>
Delivered-To: mailing list dev@struts.apache.org
Received: (qmail 572 invoked by uid 99); 12 Aug 2005 03:06:12 -0000
Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49)
    by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 11 Aug 2005 20:06:12 -0700
X-ASF-Spam-Status: No, hits=0.0 required=10.0
	tests=
X-Spam-Check-By: apache.org
Received-SPF: neutral (asf.osuosl.org: local policy)
Received: from [68.230.241.29] (HELO fed1rmmtao10.cox.net) (68.230.241.29)
    by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 11 Aug 2005 20:06:33 -0700
Received: from imbrium1 ([68.2.84.26]) by fed1rmmtao10.cox.net
          (InterMail vM.6.01.04.00 201-2131-118-20041027) with SMTP
          id <20050812030609.VQHO1860.fed1rmmtao10.cox.net@imbrium1>
          for <dev@struts.apache.org>; Thu, 11 Aug 2005 23:06:09 -0400
Message-ID: <00f101c59eea$bdf01190$040ea8c0@imbrium1>
From: "Wendy Smoak" <java@wendysmoak.com>
To: "Struts Developers List" <dev@struts.apache.org>
References: <01f201c59c84$6db7def0$040ea8c0@imbrium1>
 <f8b39ace050808190330162632@mail.gmail.com>
Subject: Re: struts-faces won't compile
Date: Thu, 11 Aug 2005 20:04:24 -0700
MIME-Version: 1.0
Content-Type: text/plain;
	format=flowed;
	charset="iso-8859-1";
	reply-type=original
Content-Transfer-Encoding: 7bit
X-Priority: 3
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook Express 6.00.2900.2670
X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.2670
X-Virus-Checked: Checked by ClamAV on apache.org
X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N

From: "Craig McClanahan" <craigmcc@gmail.com>

>> +import org.apache.struts.util.ModuleUtils;
>> -RequestUtils.selectModule(request, servletContext);
>> +ModuleUtils.getInstance().selectModule(request,servletContext);

> Conceptually that fix makes sense in that it solves the compilation
> problem ... but I don't believe that struts-faces should inherit a
> dependency on 1.3.x.

I don't see how switching from a deprecated method call to the replacement
causes a dependency on 1.3.x.  That change still compiles under 1.2.2.

> Today it works (runtime) on 1.1 and 1.2,  and it
> would be pretty pointless to throw away the vast majority of the
> potential target market.

Does it really still work on 1.1?  It won't compile against 1.1 due to the
use of org.apache.struts.taglib.TagUtils and
org.apache.struts.util.ModuleUtils, which must not have existed then.

> I think Struts-Faces (if it's going to be built with Maven :-) needs
> its own set of dependencies, independent of whatever Struts Core is
> using.

It was inheriting from the Struts Common Build, not from Struts Core.  The
incorrect dependency on struts-core-1.3.0-dev was actually *in* the
struts-faces build file and has been corrected to struts-1.2.2.  (You said
you're using 1.2.6 for the Ant build, but that one isn't on ibiblio.  Can we
agree on 1.2.7 by any chance?)

James, can you comment on how you'd like to see this handled?  The website 
builds just fine whether or not the <extend> tag is there (so I'm happy
either way).  But there are things in the common build file that are *not* 
common to all subprojects.  Maven is smart enough to allow sub-projects to
override dependencies by specifying a different version number, 
(servletapi-2.2 instead of servletapi-2.3, for example.)  But by extending 
the common build, Struts Faces picks up dependencies on antlr, 
commons-chain, commons-digester, commons-fileupload and oro, none of which 
it really needs.  I don't see a way to override a dependency coming from the 
common build with "nothing".

Thanks,
-- 
Wendy Smoak



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org