[ 
https://issues.apache.org/jira/browse/PIG-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14191399#comment-14191399
 ] 

Rohini Palaniswamy commented on PIG-4258:
-----------------------------------------

Few comments:
 - why touch in +cat touch /bin/pig_test_file ? cat tries to just check for a 
file called touch. Isn't cat /bin/pig_nonexistent_file good?
 - Newly added <property name="pig.jar.dir" value="${pig.dir}" /> not used 
anywhere. Can be removed.

> Fix several e2e tests on Windows
> --------------------------------
>
>                 Key: PIG-4258
>                 URL: https://issues.apache.org/jira/browse/PIG-4258
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>            Reporter: Daniel Dai
>            Assignee: Daniel Dai
>             Fix For: 0.14.0
>
>         Attachments: PIG-4258-1.patch
>
>
> Several issues found in Windows e2e tests:
> 1. Inconsistency in existing conf file, tmpPath is tmp/pigtest in 
> default.conf, but /tmp/pigtest in others; We do perl cleanup using "$me =~ 
> s/[^a-zA-Z0-9]*//g" in default.conf, but "chomp $me" in rpm.conf
> 2. build.xml only test tarball install, for rpm install which does not have 
> the whole tarball, compiling test udf fail
> 3. Some tests fail due to different OS error message in grunt.conf
> 4. Windows cmd such as perl/awk use a different quoting to pass parameters



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to