Mailing-List: contact dev-help@myfaces.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "MyFaces Development" <dev@myfaces.apache.org>
Received-SPF: pass (athena.apache.org: domain of lu4242@gmail.com designates
 209.85.160.53 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <1335342327.39980.YahooMailNeo@web171501.mail.ir2.yahoo.com>
References: 
 <CABujnVdFw0EaBykEkv3v5MBvdm6EPW618DZvzimPRg9qb_XKaw@mail.gmail.com>
	<1335342327.39980.YahooMailNeo@web171501.mail.ir2.yahoo.com>
Date: Wed, 25 Apr 2012 08:32:48 -0500
Message-ID: 
 <CABujnVezGREG0kuQmo=Fdgv3AfSCobsDnr4YH9VJM46Kpa2TzA@mail.gmail.com>
Subject: Re: [VOTE] release for myfaces archetypes 1.0.4
From: Leonardo Uribe <lu4242@gmail.com>
To: MyFaces Development <dev@myfaces.apache.org>,
 Mark Struberg <struberg@yahoo.de>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Hi Mark

2012/4/25 Mark Struberg <struberg@yahoo.de>:
> Hi Leo!
>
> I tried it out. Here are a few things I found out.
>

Thanks for take a look in deep into this.

> a.)
>
> The archetype catalog on your people account doesn't work.
>
> I've now used the one from the staging repo:
>
> mvn archetype:generate -DarchetypeCatalog=3Dhttps://repository.apache.org=
/content/repositories/orgapachemyfaces-096/
>

I see. It is ok. The copy of
http://people.apache.org/~lu4242/m2_archetypes_104_release/archetype-catalo=
g.xml
is to prevent loose that file when nexus deploy the files to the main
repo. That file will replace the one in
http://myfaces.apache.org/archetype-catalog.xml

>
> b.)
>
> tomcat7 integration: I know the tomcat7-maven-plugin is currently not per=
fect as it always only uses tc-7.0.25.
> For changing this you need to add a <pluginManagement> section to the pom=
 and define quite a few tc dependencies with ${tomcat7.version} or so. I kn=
ow that this sucks and I already told it to some tomcat PMC. Maybe they wil=
l release the tomcat-maven-plugin as part of the tc build in the future.
>
>
> So what do we do? Keep the pom minimal and take the out-of-the-box tc-7.0=
.25? Or should we 'pollute' the pom with the pluginManagement section?
>

I think it is ok to let it as is. The default container is jetty 8,
because it is the one that works well in different configurations
(including CODI). For most JSF applications, it will work well.

>
> c.)
>
> what's the reason to have the containers switche via -D and activation (c=
omplicated) instead of just adding a profile -Ptc ?
>

The consideration is -D could be in the future allow multiple profile
activation. See http://jira.codehaus.org/browse/MNG-3328  . If some
day that happen, we can reduce the size of the poms, because we can
use a profile for add jars as dependencies and then in the activation
section indicate the containers where this should be activated. If we
are going to unify how use the archetypes, the option using -D sounds
better, even if we have used -PjettyConfig or -Ptc for a long time. To
reduce the complexity, the instructions were added at the to of all
pom.xml files.

>
> d.)
>
> The CODI sample app crashes with the following exception after entering a=
 name and clicking on the button:
>
> javax.el.PropertyNotFoundException: Target Unreachable, identifier 'hello=
World' resolved to null
>
> viewId=3D/helloWorld.xhtml
> location=3D/Users/struberg/tmp/delete/myfacesarchetypes/mf-test-codi/src/=
main/webapp/helloWorld.xhtml
> phaseId=3DPROCESS_VALIDATIONS(3)
>
> Caused by:
> javax.el.PropertyNotFoundException - Target Unreachable, identifier 'hell=
oWorld' resolved to null
> at org.apache.el.parser.AstValue.getTarget(AstValue.java:98)
>
> /helloWorld.xhtml at line 35 and column 84 =A0 =A0value=3D"#{helloWorld.n=
ame}"
>
>
> Thus a -1 as we should fix the codi app crash in a 2nd iteration.
>

I know that. This is caused when running:

mvn clean -Dcontainer=3Dtomcat7 tomcat:run

but if you run:

mvn clean -Dcontainer=3Dtomcat7 tomcat:run-war

That's the reason why jetty 8 still is the default (the plugin has a
fix sended long time ago to detect files in /WEB-INF/lib).

To overcome that limitation, MyFaces Core has a web config param:

org.apache.myfaces.annotation.SCAN_PACKAGES

This one allow to scan for JSF annotations on directly on the relevant
packages. But CODI does not have anything similar.

In conclusion, we can't fix CODI / tomcat:run case without change its
internals, and that means another release. Please note:

mvn clean jetty:run

works without problem.

regards,

Leonardo Uribe

>
> txs and LieGrue,
> strub
>
>
>
> ----- Original Message -----
>> From: Leonardo Uribe <lu4242@gmail.com>
>> To: MyFaces Development <dev@myfaces.apache.org>
>> Cc:
>> Sent: Wednesday, April 25, 2012 12:24 AM
>> Subject: [VOTE] release for myfaces archetypes 1.0.4
>>
>> Hi,
>>
>> I was running the needed tasks to get the 1.0.4 release of Apache
>> MyFaces Build Tools Archetypes out.
>>
>> Please note that this vote concerns all of the following parts:
>> 1. Maven artifact group "org.apache.myfaces.buildtools" v1.0.4 (only
>> archetypes)
>>
>> The artifacts are deployed to the nexus repository [1].
>>
>> The following issues have been addressed in this release:
>>
>> * Update archetypes to recent versions of MyFaces Core, CODI, OWB.
>> * Include tomcat7-maven-plugin reference into archetype
>> * Update JSF 2.0 archetypes to jetty 8
>> * Add myfaces-helloworld20-debug archetype
>>
>> To test the archetypes just do the following:
>>
>> Create a project from an archetype:
>>
>> mvn archetype:generate
>> -DarchetypeCatalog=3Dhttp://people.apache.org/~lu4242/m2_archetypes_104_=
release
>>
>> Choose an archetype from the list and enter values for groupId,
>> artifactId and version.
>>
>> Then on the path of the generated archetype
>>
>> mvn clean jetty:run-exploded
>>
>> or for the 2.0 archetypes
>>
>> mvn clean jetty:run-exploded -PjettyConfig
>>
>> =A0 =A0 =A0 - Run using jetty
>> =A0 =A0 =A0 =A0 =A0 mvn clean jetty:run
>> =A0 =A0 =A0 =A0 =A0 mvn clean -Dcontainer=3Djetty-mojarra jetty:run
>> =A0 =A0 =A0 =A0 =A0 Set your browser to http://localhost:8080/${artifact=
Id}
>> =A0 =A0 =A0 - Run using tomcat
>> =A0 =A0 =A0 =A0 =A0 mvn clean -Dcontainer=3Dtomcat7 tomcat:run
>> =A0 =A0 =A0 =A0 =A0 Set your browser to http://localhost:8080/${artifact=
Id}
>> =A0 =A0 =A0 - Run war using tomcat
>> =A0 =A0 =A0 =A0 =A0 mvn clean -Dcontainer=3Dtomcat7 tomcat:run-war
>> =A0 =A0 =A0 =A0 =A0 Set your browser to http://localhost:8080/${artifact=
Id}
>> =A0 =A0 =A0 - Run war using jetty (remove <webApp> config inside plugin =
config
>> first)
>> =A0 =A0 =A0 =A0 =A0 mvn clean jetty:run-war
>> =A0 =A0 =A0 =A0 =A0 mvn clean -Dcontainer=3Djetty-mojarra jetty:run-war
>> =A0 =A0 =A0 =A0 =A0 Set your browser to http://localhost:8080
>> =A0 =A0 =A0 - Build war and bundle JSF jars
>> =A0 =A0 =A0 =A0 =A0 mvn clean -Pbundle-myfaces install
>> =A0 =A0 =A0 =A0 =A0 mvn clean -Pbundle-mojarra install
>>
>> Please take a look at the "1.0.4" artifacts and vote!
>>
>> Please note: This vote is "majority approval" with a minimum of three
>> +1 votes (see [2]).
>>
>> ------------------------------------------------
>> [ ] +1 for community members who have reviewed the bits
>> [ ] +0
>> [ ] -1 for fatal flaws that should cause these bits not to be released,
>> and why..............
>> ------------------------------------------------
>>
>> Thanks,
>> Leonardo
>>
>> [1] https://repository.apache.org/content/repositories/orgapachemyfaces-=
096/
>> [2] http://www.apache.org/foundation/voting.html#ReleaseVotes
>>