[Trinidad] JSF2: Facelet tag metadata

2010-02-04 Thread Max Starets
Hello Everyone, Does anybody have any objections to enhancing Trinidad maven plugins to write out Facelet tag metadata in taglib xml (see https://issues.apache.org/jira/browse/TRINIDAD-1707)? I have the implementation ready. Thanks, Max

Re: [Trinidad] JSF2: Facelet tag metadata

2010-02-04 Thread Blake Sullivan
+1 -- Blake Sullivan Max Starets said the following On 2/4/2010 7:57 AM PT: Hello Everyone, Does anybody have any objections to enhancing Trinidad maven plugins to write out Facelet tag metadata in taglib xml (see https://issues.apache.org/jira/browse/TRINIDAD-1707)? I have the

Re: [Trinidad] JSF2: Facelet tag metadata

2010-02-04 Thread Matthias Wessendorf
+1 - yeah would be nice as the plugins cover already faces cfg part for that. So, feel free to upload the patch, since the impl is done :-) -M On Thu, Feb 4, 2010 at 5:57 PM, Blake Sullivan blake.sulli...@oracle.com wrote: +1 -- Blake Sullivan Max Starets said the following On 2/4/2010

Re: [Trinidad] JSF2: Facelet tag metadata

2010-02-04 Thread Matthias Wessendorf
Hello Max, shouldn't we check for the jsfVersion, for the metadata ? -Matthias On Thu, Feb 4, 2010 at 6:08 PM, Matthias Wessendorf mat...@apache.org wrote: +1 - yeah would be nice as the plugins cover already faces cfg part for that. So, feel free to upload the patch, since the impl is done

Re: [Trinidad] JSF2: Facelet tag metadata

2010-02-04 Thread Max Starets
Matthias, What do you mean? The change is going only to the 2.0.x branch. It will only be used with JSF 2.0+. Max Matthias Wessendorf wrote: Hello Max, shouldn't we check for the jsfVersion, for the metadata ? -Matthias On Thu, Feb 4, 2010 at 6:08 PM, Matthias Wessendorf

Re: [Trinidad] JSF2: Facelet tag metadata

2010-02-04 Thread Matthias Wessendorf
Well, there is only one version of the plugin. Trunk covers JSF 1.1 and 1.2 The goal is to continue that as well. The branch was just created to have the (experimental) JSF2 development not mixed with those changes on trunk. Do you really feel a need to introduce again a 2-way plugin ? a) 1.2

Re: [Trinidad] JSF2: Facelet tag metadata

2010-02-04 Thread Max Starets
Matthias, I was fooled by the fact that it was a '2.0.0.x' branch. I have no problem adding a check. Max Matthias Wessendorf wrote: Well, there is only one version of the plugin. Trunk covers JSF 1.1 and 1.2 The goal is to continue that as well. The branch was just created to have the

Re: [Trinidad] JSF2: Facelet tag metadata

2010-02-04 Thread Matthias Wessendorf
:-) I am not sure if that makes sense. I felt it did, when I added the first check for JSF2. I didn't want to mess around w/ trunk for the first Trindiad2 added plugin features Does that make sense? -Matthias On Thu, Feb 4, 2010 at 9:42 PM, Max Starets max.star...@oracle.com wrote: Matthias,

Re: [Trinidad] JSF2: Facelet tag metadata

2010-02-04 Thread Max Starets
Yes. Matthias Wessendorf wrote: :-) I am not sure if that makes sense. I felt it did, when I added the first check for JSF2. I didn't want to mess around w/ trunk for the "first" Trindiad2 added plugin "features" Does that make sense? -Matthias On Thu, Feb 4, 2010 at 9:42 PM, Max