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

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

Hi Forrest,

That definitely fixed the line-ending issues.  However, I think we still need 
some more work to this patch before we apply it. 
It is named as being a web-only version of daytrader but it is also tomcat 
specific.  There was discussion when this was first proposed that we should 
support both jetty and tomcat.  If we cannot have a single generic version than 
we should perhaps create two webonly versions - one for tomcat and one for 
jetty.   I wonder if we can configure this something like the console so that 
we can have a generic war and then tomcat and jetty specific plugins for easy 
installation.  We need to think about this some more ... but I wanted to post 
the concern so you were aware of the issue.

Thanks, Joe

> 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: Forrest Xia
>         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