[ 
https://issues.apache.org/jira/browse/DAYTRADER-68?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12709873#action_12709873
 ] 

Joe Bohn commented on DAYTRADER-68:
-----------------------------------

I decided to go ahead and checkin the code so that further collaboration could 
take place in svn.  I added the missing headers, fixed the build, included 
commons-logging in the war (not included in tomcat 6 and decided it was better 
in the war than manually pulling it into tomcat) and beautified some code.

Included in trunk rev. 775187

Still needed is to create plugins for Geronimo jetty and tomcat.   We also need 
to reconcile the datasource issues for Geronimo (perhaps we can share a common 
datasource with the full sample).

> A version of daytrader for Tomcat 6
> -----------------------------------
>
>                 Key: DAYTRADER-68
>                 URL: https://issues.apache.org/jira/browse/DAYTRADER-68
>             Project: DayTrader
>          Issue Type: New Feature
>      Security Level: public(Regular issues) 
>          Components: Web Tier
>    Affects Versions: 2.2
>         Environment: Tomcat 6.0.18
>            Reporter: Forrest Xia
>            Assignee: Joe Bohn
>             Fix For: 2.2
>
>         Attachments: daytrader-webonly-new.patch, daytrader-webonly-new.zip, 
> daytrader-webonly.patch
>
>
> I have a trimmed version of daytrader for tomcat 6, and want to contribute to 
> daytrader trunk, please help review and commit. thanks!

-- 
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