[ 
https://issues.apache.org/jira/browse/MYFACES-2504?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12829743#action_12829743
 ] 

Matthias Weßendorf commented on MYFACES-2504:
---------------------------------------------

OK,

so the "final" patch is => "2504-3.diff" ?

Here are some comments.
* StartupServletContextListener:
+            getFacesInitializer(_servletContext).initFaces(_servletContext);
Doesn't that look odd ? Hrm..

* ContainerUtils:
=> .startsWith("Google App Engine"))
OK, but I wonder if they have some sort of guarantee for that ?

=> the isRunningOnGoogleAppEngine() has some duplicated code...
Also regarding getServerInfo() I filed this:
https://javaserverfaces-spec-public.dev.java.net/issues/show_bug.cgi?id=739


Doc patch is OK. what is the license for the "jdo2-api-2.3-eb.jar" JAR? 
Datanuculus is Apache.
(cool plugin btw)

A demo app would be cool :-) 


> Google App Engine Support
> -------------------------
>
>                 Key: MYFACES-2504
>                 URL: https://issues.apache.org/jira/browse/MYFACES-2504
>             Project: MyFaces Core
>          Issue Type: Improvement
>          Components: JSR-252, JSR-314
>    Affects Versions: 1.2.8, 2.0.0-alpha
>         Environment: Google App Engine 1.3
>            Reporter: Ali Ok
>            Priority: Minor
>         Attachments: 2504-2.diff, 2504-3.diff, 2504-doc.diff, 2504.diff
>
>
> Support for Google App Engine for MyFaces 1.2 and 2.0.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to