geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joe Bohn (JIRA)" <>
Subject [jira] Commented: (DAYTRADER-68) A version of daytrader for Tomcat 6
Date Tue, 12 May 2009 22:11:45 GMT


Joe Bohn commented on DAYTRADER-68:

Some more comments and questions:
- I noticed that ./daytrader-webonly/daytrader-datasource-plan.xml and ./daytrader-webonly/src/main/webapp/META-INF/context.xml
are missing the Apache License header.  Can you please add them with your contribution?
-  We probably need some more description (in the README?) about the datasource and how to
handle this for Geronimo installations and possibly Tomcat installs as well.  I can see where
it might be confusing with the daytrader-derby-datasource that we also have out there.  Perhaps
we need to include webonly in this name as well.
- were you working on the tomcat/jetty plugins of this that were discussed on the original

> A version of daytrader for Tomcat 6
> -----------------------------------
>                 Key: DAYTRADER-68
>                 URL:
>             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: Forrest Xia
>         Attachments: daytrader-webonly-new.patch,, 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.

View raw message