Return-Path: Delivered-To: apmail-myfaces-dev-archive@www.apache.org Received: (qmail 77789 invoked from network); 24 Mar 2011 23:01:28 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 24 Mar 2011 23:01:28 -0000 Received: (qmail 13308 invoked by uid 500); 24 Mar 2011 23:01:28 -0000 Delivered-To: apmail-myfaces-dev-archive@myfaces.apache.org Received: (qmail 13268 invoked by uid 500); 24 Mar 2011 23:01:28 -0000 Mailing-List: contact dev-help@myfaces.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "MyFaces Development" Delivered-To: mailing list dev@myfaces.apache.org Received: (qmail 13261 invoked by uid 99); 24 Mar 2011 23:01:28 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 24 Mar 2011 23:01:28 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of lu4242@gmail.com designates 209.85.161.53 as permitted sender) Received: from [209.85.161.53] (HELO mail-fx0-f53.google.com) (209.85.161.53) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 24 Mar 2011 23:01:21 +0000 Received: by fxm11 with SMTP id 11so603244fxm.12 for ; Thu, 24 Mar 2011 16:01:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=l7qM7sU9y3hN36CX32Zxijk9k4kbM0HMbSAF/6PGHws=; b=c5qZnojYS0F/v83X6TKZqco3ef0+sm/kyZFNh8pwB8g+JK+lGNaAlQxpeYnD+psrqI PSZv/rGXbnM5N7xLZH407BRsyYk8VLNuE4afYkXQCk893zpa1NZry5v48OeHvddh3+/s i1Ls/50YjAOlR93TuFy5eN3tF+MwmitOQofaU= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=l3tS37kpvcb9phRcycUsoCMQOz730ZYXcWK38w6tJoVWc9qpIgLiNUe5bxEjGIHy1n BtLQcBjQCD8aalUq6o+iswCRkPiocOUMquL5rc7/1tqQRllfTUcGNAKBfoU/7b4vTCLr gpguxOYZLKt2cgsJnSgKk5Nqxua/iZOaiyb10= MIME-Version: 1.0 Received: by 10.223.87.68 with SMTP id v4mr59391fal.116.1301007661185; Thu, 24 Mar 2011 16:01:01 -0700 (PDT) Received: by 10.223.74.200 with HTTP; Thu, 24 Mar 2011 16:01:01 -0700 (PDT) In-Reply-To: <3737004005692877407@unknownmsgid> References: <4D8B722E.3080805@gmail.com> <4D8B9E5F.9020709@oracle.com> <4D8BB123.2040105@oracle.com> <4D8BB92B.10504@oracle.com> <4D8BBC5A.9010604@oracle.com> <3737004005692877407@unknownmsgid> Date: Thu, 24 Mar 2011 18:01:01 -0500 Message-ID: Subject: Re: [VOTE] Release of Trinidad Maven Plugins v. 2.0.5 From: Leonardo Uribe To: MyFaces Development Cc: Andy Schwartz Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable X-Virus-Checked: Checked by ClamAV on apache.org +1 2011/3/24 Scott O'Bryan : > Is that a +1? =A0:D > > On Mar 24, 2011, at 4:06 PM, Leonardo Uribe wrote: > >> Hi >> >> I have created these issues: >> >> http://java.net/jira/browse/JAVASERVERFACES_SPEC_PUBLIC-952 >> >> https://issues.apache.org/jira/browse/MYFACES-3082 >> >> I agree with the proposed behavior, and I don't think do it could >> cause any problems. So from my side there is no objections about the >> artifacts proposed. Thanks for the clarification. >> >> Leonardo >> >> 2011/3/24 Andy Schwartz : >>> Gang - >>> >>> Looking back at the EG emails, I realize now that I dropped the ball >>> on making sure that my proposed changes actually made it into the >>> spec. >>> >>> Here was my original email ("Metadata complete jar files") from >>> Septeber 3, 2009: >>> >>>> Gang - >>>> >>>> Section 11.5.1 of the spec defines the following annotation scanning b= ehavior: >>>> >>>>> Requirements for scanning of classes for annotations >>>>> * If the element in the WEB-INF/faces-config.xml file = contains metadata-complete attribute whose value is =93true=94, the impleme= ntation must not perform annotation scanning on any classes except for thos= e classes provided by the implementation itself. Otherwise, continue as fol= lows. >>>>> * If the runtime discovers a conflict between an entry in the Applica= tion Configuration Resources and an annotation, the entry in the Applicatio= n Configuration Resources takes precedence. >>>>> * All classes in WEB-INF/classes must be scanned. >>>>> * For every jar in the application's WEB-INF/lib directory, if the ja= r contains a =93META-INF/faces-config.xml=94 file or a file that matches th= e regular expression =93.*\.faces-config.xml=94 (even an empty one), all cl= asses in that jar must be scanned. >>>> >>>> >>>> Since application developers have the ability to disable annotation sc= anning at a global level, this means that any component set that wants to s= upport this mode would need to provide a metadata complete faces-config.xml= file. I don't think this is a hardship for most component vendors, since p= resumably component vendors are going to want to provide design-time metada= ta (eg. JSR-276 metadata), which, for the moment, requires a faces-config.x= ml file anyway. >>>> >>>> A question that came up here is whether we can tweak section 11.5.1 to= accommodate metadata complete jar files. That is, can we specify that any = jar that contains a faces-config.xml with a metadata-complete=3D"true" attr= ibute would not be scanned? This would allow component vendors to indicate = that their jar files are metadata complete, and thus avoid the cost of anno= tation scanning for the contents of the jar. >>>> >>>> Note that while the annotation scan is typically a one time hit (durin= g application startup), design-time tools may end up starting/stopping JSF = repeatedly during the development process. Thus, avoiding unnecessary scann= ing should provide for a more efficient development experience. >>>> >>>> Any thoughts on whether we could/should make this change? Does anyone = know of reasons why we avoided specifying this originally? >>>> >>>> Also - if we agree to make this change, is this small enough that we c= ould get this into the the next MR? >>>> >>>> Andy >>> >>> Both Dan and Pete responded in support. =A0There were no other response= s >>> on the EG list. >>> >>> I should have followed up to make sure that the spec update happened, >>> but apparently never did. =A0I will do that now. :-) >>> >>> Sorry about the confusion. :-( >>> >>> Andy >>> >