[ 
https://issues.apache.org/jira/browse/THRIFT-166?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12667887#action_12667887
 ] 

David Reiss commented on THRIFT-166:
------------------------------------

What is '''+  <property file="${user.home}/.thrift-build.properties" />''' for?

I don't see any of the moved files in the patch.

> Java tests should be in lib/java/test/...
> -----------------------------------------
>
>                 Key: THRIFT-166
>                 URL: https://issues.apache.org/jira/browse/THRIFT-166
>             Project: Thrift
>          Issue Type: Sub-task
>          Components: Library (Java)
>            Reporter: Bryan Duxbury
>            Assignee: Bryan Duxbury
>            Priority: Minor
>         Attachments: thrift-166.patch
>
>
> It's confusing and inconvenient to have the code and tests in separate 
> folders. If they were together, the build.xml could allow you to build and 
> run the tests in one command without changing directories. 
> Would anyone object to moving these tests into lib/java/test? At the same 
> time, we could start doing proper packaging and JUnit naming.

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