[jira] Commented: (MYFACES-2995) Make method of determinine app context in FactoryFinder pluggable

2011-02-24 Thread David Jencks (JIRA)
[ https://issues.apache.org/jira/browse/MYFACES-2995?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12999056#comment-12999056 ] David Jencks commented on MYFACES-2995: --- This looks ok to me although I don't like

[jira] Commented: (MYFACES-2995) Make method of determinine app context in FactoryFinder pluggable

2011-02-19 Thread David Jencks (JIRA)
[ https://issues.apache.org/jira/browse/MYFACES-2995?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12996947#comment-12996947 ] David Jencks commented on MYFACES-2995: --- Since it's been about 2 1/2 months before

Re: Issue 2995 - Leo please read

2011-02-11 Thread David Jencks
that putting it in myfaces impl is not a good idea because theoretically myfaces-api can be used with another jsf implementation. thanks david jencks regards, Leonardo Uribe

Re: [VOTE] release for myfaces core 2.0.4

2011-02-09 Thread David Jencks
this suggestion? Out of curiosity, how are myfaces releases scheduled? I didn't see any notice or warning on the dev list that this vote was imminent. thanks david jencks On Feb 8, 2011, at 10:07 PM, Leonardo Uribe wrote: Hi, I was running the needed tasks to get the 2.0.4 release of Apache

Re: Issue 2995 - Leo please read

2011-02-09 Thread David Jencks
+ impl bundle), so I'd be interested to know what problems Leonardo has run into. And, sorry for sounding disgruntled I didn't realize 2.0.4 was a kind of emergency bug fix release. thanks david jencks On Feb 9, 2011, at 6:44 AM, Werner Punz wrote: If it is for OSGI only I think

Re: Classloaders and component boundaries

2011-01-11 Thread David Jencks
Hi, It's a month later, holidays are over, I'd like to ask again that this be considered. thanks david jencks On Dec 6, 2010, at 6:09 PM, Leonardo Uribe wrote: Hi I don't think we can include it for 2.0.3 release. This change should be discussed carefully, and that requires some time

Re: Classloaders and component boundaries

2011-01-11 Thread David Jencks
carefully at the ee spec and think that 1 classloader per ear is definitely allowed. thanks david jencks So imo this sounds like a no-go LieGrue, strub --- On Tue, 12/7/10, David Jencks david_jen...@yahoo.com wrote: From: David Jencks david_jen...@yahoo.com Subject: Classloaders

Re: ordering tck failures in geronimo

2010-12-07 Thread David Jencks
for your response) . 2010/12/7 David Jencks david_jen...@yahoo.com I have not compared the myfaces classes with the openejb jaxb tree for faces-config.xml, so I have no idea how plausible this idea might be. Would it be possible to annotate the myfaces object tree representing faces

Re: ordering tck failures in geronimo

2010-12-06 Thread David Jencks
the default implementation of the spi then there is something wrong with the interface design. If the sorting is universal then perhaps it should not be in a class implemented by a service provider? thanks david jencks -- Ivan

[jira] Created: (MYFACES-2995) Make method of determinine app context in FactoryFinder pluggable

2010-12-06 Thread David Jencks (JIRA)
: Improvement Components: General Affects Versions: 2.0.3-SNAPSHOT Reporter: David Jencks As discussed on dev list geronimo would like to explicitly mark component boundaries rather than relying on the TCCL changing. -- This message is automatically generated by JIRA

Classloaders and component boundaries

2010-12-06 Thread David Jencks
this proposal out for consideration and review ASAP. I've opened MYFACES-2995 and am attaching an initial patch for consideration. thanks david jencks

[jira] Updated: (MYFACES-2995) Make method of determinine app context in FactoryFinder pluggable

2010-12-06 Thread David Jencks (JIRA)
[ https://issues.apache.org/jira/browse/MYFACES-2995?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jencks updated MYFACES-2995: -- Status: Patch Available (was: Open) Make method of determinine app context

ordering tck failures in geronimo

2010-12-04 Thread David Jencks
, expertise, and a few minutes could take a look and provide some advice I'd really appreciate it. thanks david jencks

Possible problem in FacesConfig ordering

2010-12-03 Thread David Jencks
()) assuming a non-null argument. Whatever the cause of the null I'm seeing returned from getFacesConfig this doesn't seem right. thanks david jencks

Re: [core] tasks for release myfaces core 2.0.3

2010-12-01 Thread David Jencks
to be filing jiras for the other issues tomorrow. thanks! david jencks On Nov 30, 2010, at 6:23 PM, Leonardo Uribe wrote: Hi It could be good to do a release of myfaces core 2.0.3 next week, after release myfaces-builder-plugin. The last patch required to add support to myfaces with geronimo

[jira] Created: (MYFACES-2991) NPE from rendering button with missing resource URL

2010-12-01 Thread David Jencks (JIRA)
: General Affects Versions: 2.0.3-SNAPSHOT Reporter: David Jencks Attachments: MYFACES-2991.patch I ran into this with the tck in geronimo. The test appears to be trying to render a button that is just created and not part of a view to test the EL bits of encodeBegin

Re: [core] tasks for release myfaces core 2.0.3

2010-12-01 Thread David Jencks
Thanks for taking care of MYFACES-2983. I've opened MYFACES-2990 and MYFACES-2991 for the other two problems I know about. At this point these are the only critical problems for geronimo that I know about although I may have some further suggestions in a couple days. many thanks! david

Compile problem in core?

2010-11-30 Thread David Jencks
org.apache.myfaces.config.FacesConfigurator ... (several more) Anyone else seeing this? I think the relevant methods were removed from FacesConfigurator in rev 1039554 thanks david jencks

[jira] Created: (MYFACES-2983) Consider using myfaces class's classloader when loading resources

2010-11-29 Thread David Jencks (JIRA)
: Bug Components: General Affects Versions: 2.0.3-SNAPSHOT Reporter: David Jencks There are a few instances where only the TCCL is considered for loading resources that are actually in a myfaces jar. This won't work in an isolated classes environment like OSGI. A simple

Re: Real world MyFaces 2.0 webapp

2010-11-28 Thread David Jencks
3 lite already. Or you might try packaging it in two ways, one as a web profile app and one as a servlet app with all the extra support included in the war. david jencks BTW, it's not a *real* real-world app (it's not running in production for a client). I'm not allowed to do

Re: Real world MyFaces 2.0 webapp

2010-11-27 Thread David Jencks
and OpenJPA 2.0. I also use Hibernate Validator. If you want to line up with more apache projects you could look into apache bval. david jencks I got a strange error with CODI, so I skipped it for now to stay up-to-speed, but I guess we should get that one in. My goal with this example app

[jira] Commented: (MYFACES-2976) Support hiding myfaces impl classes in osgi, and provide a single osgi bundle for api + impl

2010-11-26 Thread David Jencks (JIRA)
[ https://issues.apache.org/jira/browse/MYFACES-2976?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12936113#action_12936113 ] David Jencks commented on MYFACES-2976: --- I think the downloaded snapshot is working

[jira] Commented: (MYFACES-2976) Support hiding myfaces impl classes in osgi, and provide a single osgi bundle for api + impl

2010-11-25 Thread David Jencks (JIRA)
[ https://issues.apache.org/jira/browse/MYFACES-2976?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12935946#action_12935946 ] David Jencks commented on MYFACES-2976: --- Jakob's revised patch seems to work fine

[jira] Commented: (MYFACES-2976) Support hiding myfaces impl classes in osgi, and provide a single osgi bundle for api + impl

2010-11-18 Thread David Jencks (JIRA)
[ https://issues.apache.org/jira/browse/MYFACES-2976?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12933662#action_12933662 ] David Jencks commented on MYFACES-2976: --- I added the export

[jira] Created: (MYFACES-2976) Support hiding myfaces impl classes in osgi, and provide a single osgi bundle for api + impl

2010-11-17 Thread David Jencks (JIRA)
Project: MyFaces Core Issue Type: Improvement Components: Extension Feature Affects Versions: 2.0.3-SNAPSHOT Reporter: David Jencks The current osgi solution of using require-bundle between the api and impl jars is generally thought of as osgi worst

Re: require-bundle header in api bundle

2010-11-08 Thread David Jencks
. thanks david jencks On Nov 8, 2010, at 10:25 AM, Leonardo Uribe wrote: Hi 2010/11/5 David Jencks david_jen...@yahoo.com I'm trying to understand why the require-bundle header is in the api bundle. This is a pretty bizarre thing to do in what I've seen of osgi. Could you be more

Re: Use maven-shade-plugin to prevent duplicate code

2010-11-08 Thread David Jencks
If you don't need a shaded source jar, have you guys experimented with the bundle plugin for generating this jar? thanks david jencks On Nov 8, 2010, at 2:15 PM, Leonardo Uribe wrote: Hi No, I understand well, but maybe we are talking about many possible enhancements at the same time

[jira] Created: (MYFACES-2964) Incorect osgi Import-Package in impl

2010-11-06 Thread David Jencks (JIRA)
Reporter: David Jencks myfaces will only run on tomcat in osgi due to a mistaken Import-Package entry generated due to tomcat integration code. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.

Re: [CORE] 2.0.3 release ?

2010-11-06 Thread David Jencks
doesn't work with snapshot versions. many thanks! david jencks On Nov 6, 2010, at 7:53 AM, Leonardo Uribe wrote: Hi I think we need to do the integration with geronimo first. The only thing left is an interface for cache faces-config.xml information. I hope to do it on the following days

require-bundle header in api bundle

2010-11-05 Thread David Jencks
, but maybe not. I saw some vague claims in the comments to MYFACES-2911 that the api classes need to load impl classes, but no details. Could someone point me to the code where this happens? There must be a better way to do this thanks david jencks

Re: [maven] myfaces pom

2010-11-05 Thread David Jencks
the plugin is released. I'd include a comment on why its needed too. thanks david jencks On Nov 5, 2010, at 1:49 AM, Matthias Wessendorf wrote: Hi, the ASF master-pom does contain a _regular_ snapshot repo (linked to the (new) nexus repo), see: https://svn.apache.org/repos/asf/maven/pom/trunk

Re: [VOTE] release of myfaces core 2.0.0-alpha

2009-12-26 Thread David Jencks
Many thanks for applying this! If it doesn't go against any myfaces development policies, it would be great if someone could deploy a snapshot built after this patch. thanks david jencks On Dec 24, 2009, at 5:33 AM, Matthias Wessendorf wrote: http://svn.apache.org/viewvc?view

Re: [VOTE] release of myfaces core 2.0.0-alpha

2009-12-23 Thread David Jencks
scenario that currently works that this proposed change would affect, all it does is allow myfaces to be used in more environments. If you think this change will prevent a currently working scenario from working please explain what it is and how. thanks david jencks Scott David Jencks

Re: [VOTE] release of myfaces core 2.0.0-alpha

2009-12-21 Thread David Jencks
to MYFACES-2290 as https://issues.apache.org/jira/secure/attachment/12428613/allow-ee6-versioned-apis.diff With this patch we can at least start a server that has loaded myfaces 2. Hopefully soon we'll be able to run the ee6 version of the tck. many thanks david jencks On Nov 26, 2009, at 6:23 AM

[jira] Commented: (MYFACES-1761) Handling PostConstruct annotations - wrong order

2007-11-06 Thread David Jencks (JIRA)
[ https://issues.apache.org/jira/browse/MYFACES-1761?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12540615 ] David Jencks commented on MYFACES-1761: --- I'm not sure you have your environment configured correctly

Re: [jira] Updated: (MYFACES-1559) Proposal for dependency injection and lifecycle method handling

2007-07-18 Thread David Jencks
Isn't this complete? We're happy in geronimo with how it's working. thanks david jencks On Jul 17, 2007, at 11:06 PM, Matthias Weßendorf (JIRA) wrote: [ https://issues.apache.org/jira/browse/MYFACES-1559? page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias

Re: SUN PROPRIETARY/CONFIDENTIAL code in myfaces

2007-06-20 Thread David Jencks
improvement IMO. many thanks, david jencks On Jun 20, 2007, at 9:31 PM, Craig L Russell wrote: I've been working with Sun to get the appropriate legal notices changed in the relevant files: the xsd for faces 1.2 and the dtd for faces 1.0 and 1.1. Please take a look at the newly updated files

Re: Tomahawk, Myfaces 1.2 and Tomcat 6

2007-05-21 Thread David Jencks
explain the differences. thanks david jencks Cheers, Bruno On 21/05/07, Martin Haimberger [EMAIL PROTECTED] wrote: Hi *, anyone running Tomahawk with MyFaces 12 and Tomcat 6 successfully? I have created the intellij project with the -Djsf=12 parameter, which is working fine for the creation

Re: MyFaces 2.0.0 (was Re: Tomahawk 1.1.5 release plans?)

2007-05-18 Thread David Jencks
it fundamentally works differently than 1.1 then 2.0 (or higher :-) would be more appropriate. I have pretty limited exposure to myfaces but have the impression that 1.2 would be more in line with the extent and nature of the changes. thanks david jencks On May 18, 2007, at 3:14 PM, Grant Smith

Re: [PROPOSAL] Create a notifications mailing list

2007-05-15 Thread David Jencks
. Having an automated build system is a lot more reliable to catch problems than waiting for people to complain on the mailing list or trying to remember to check out the changes into another copy of the project and build it there. thanks david jencks On 5/15/07, Bruno Aranda [EMAIL

Re: Use 1.2 as current

2007-04-18 Thread David Jencks
What is in head that needs to be merged with the 1.2 branch? Why not move head to 1.1.5_1 and move 1.2 to head? I'm rather worried that the spec compliance will go down drastically if there are extensive merges. thanks david jencks On Apr 18, 2007, at 1:35 AM, Martin Marinschek wrote

Re: Are the web-facesconfig_1_0.dtd and web-facesconfig_1_1.dtd files really SUN confidential/proprietary property?

2007-04-02 Thread David Jencks
on the web without going through even a clickthrough license can be maintained to be confidential is a question I can only assume lawyers would love to answer. thanks david jencks On Apr 2, 2007, at 8:23 AM, Lance Bader wrote: I am disturbed by the web-facesconfig_1_0.dtd and web

How are snapshot jars getting deployed?

2007-03-30 Thread David Jencks
david jencks

[jira] Commented: (MYFACES-1559) Proposal for dependency injection and lifecycle method handling

2007-03-15 Thread David Jencks (JIRA)
[ https://issues.apache.org/jira/browse/MYFACES-1559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12481195 ] David Jencks commented on MYFACES-1559: --- Thanks for implementing this, I've hooked it up to geronimo

Re: Proposal for annotation processing

2007-03-13 Thread David Jencks
/ AnnotationProcessorFactory rather than having the factory examine its environment for something that might work. many thanks! david jencks On Mar 12, 2007, at 6:06 PM, Bernd Bohmann wrote: Hello David, we can move the AnnotationProcessor to the package org.apache.myfaces or an other package and add

Re: Proposal for annotation processing

2007-03-13 Thread David Jencks
in less than a day and even less confidence that it wouldn't be accidently subverted by a user app. thanks david jencks Regards Bernd Mathias Brökelmann wrote: IMO the simple interface which David suggest is quite sufficient to solve the problem. I also think that discovery to look up

Re: Proposal for annotation processing

2007-03-13 Thread David Jencks
be called on all managed beans, based on the spec language you quote thanks david jencks Regards Bernd David Jencks wrote: I actually think my MYFACES-1559-2.patch is a better solution than the current code + my MYFACES-1559-3.patch. I don't think anyone will be interested in implementing

Re: Proposal for annotation processing

2007-03-13 Thread David Jencks
On Mar 13, 2007, at 4:39 PM, Bernd Bohmann wrote: Hello David, comments inline David Jencks wrote: On Mar 13, 2007, at 3:38 PM, Bernd Bohmann wrote: I read this to mean that the jsf implementation is prohibited from calling posConstruct methdods after putting the bean in scope

Re: Servlet API thread safety ... was [jira] MYFACES-1558

2007-03-13 Thread David Jencks
the app starts and injecting them into any object that could possibly want to use them would avoid your performance concerns and clarify the structure of the framework. You may have noticed my opinion of discovery on another thread :-) thanks david jencks Dennis Byrne On 3/13/07, David

[jira] Created: (MYFACES-1559) Proposal for dependency injection and lifecycle method handling

2007-03-12 Thread David Jencks (JIRA)
Feature Components: General Affects Versions: 1.2.0-SNAPSHOT Reporter: David Jencks Fix For: 1.2.0-SNAPSHOT I'm opening this jira as a convenient place to put code samples in support of discussion on the dev list Based on my experience implementing

Proposal for annotation processing

2007-03-12 Thread David Jencks
the LifecycleProviderToAnnotationProcessorAdapter. The jira issue is https://issues.apache.org/jira/browse/MYFACES-1559 Comments? Flames? many thanks, david jencks

[jira] Created: (MYFACES-1557) MyfacesConfig.getCurrentInstance not thread safe

2007-03-11 Thread David Jencks (JIRA)
Affects Versions: 1.2.0-SNAPSHOT Reporter: David Jencks Fix For: 1.2.0-SNAPSHOT If getCurrentInstance decides to create a new MyFacesConfig, it sets it before filling in the values. This is obviously not thread safe. With the current MyFaces implementations getting

[jira] Created: (MYFACES-1558) FactoryFinder not thread safe, and has a memory leak

2007-03-11 Thread David Jencks (JIRA)
: General Affects Versions: 1.2.0-SNAPSHOT Reporter: David Jencks FactoryFinder has insufficient synchronization to be thread safe, and has a memory leak in _registeredFactoryNames. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment

[jira] Commented: (MYFACES-1558) FactoryFinder not thread safe, and has a memory leak

2007-03-11 Thread David Jencks (JIRA)
[ https://issues.apache.org/jira/browse/MYFACES-1558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12479973 ] David Jencks commented on MYFACES-1558: --- Thanks for looking into this. I have been working with what you

source-repository page confusing

2007-03-07 Thread David Jencks
hoping that the current code base is actually located at https:// svn.apache.org/repos/asf/myfaces/core/trunk/ thanks david jencks