avro-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Thiruvalluvan M. G. (JIRA)" <j...@apache.org>
Subject [jira] Commented: (AVRO-338) ant test-tools does not work under Cygwin
Date Sat, 16 Jan 2010 02:45:54 GMT

    [ https://issues.apache.org/jira/browse/AVRO-338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12801078#action_12801078
] 

Thiruvalluvan M. G. commented on AVRO-338:
------------------------------------------

Yes, in general, Python will be a much better alternative to shell.

But the specific problem here is that Java which is completely unaware of Cygwin, works with
Windows paths and any Cygwin based scripting engine will understand only Cygwin paths and
so they don't interact well. Another minor irritation is that under Windows Java's classpath
separator is semicolon and under Unix, it is colon. There are the following options:

   - Use relative paths always. Then the only difference between Cygwin and Windows paths
is forward- versus back-slash. Windows API is fortunately okay with forward slash.
   - Use cygpath Cygwin's path translation tool. This will lead to some kind "if (cygwin)"
constructs in the scripts.
   - Use ant. Since ant is a Java utility, it does the right thing for Windows.

None of these is a pretty solution. I'm leaning toward ant because, we already need ant (no
new dependency) and we have (at least for now) just one small script to translate.

> ant test-tools does not work under Cygwin
> -----------------------------------------
>
>                 Key: AVRO-338
>                 URL: https://issues.apache.org/jira/browse/AVRO-338
>             Project: Avro
>          Issue Type: Bug
>          Components: java
>            Reporter: Thiruvalluvan M. G.
>
> The target "test-tools" invokes a shell script using "exec" task. Since the shell scripts
cannot be invoked directly by the Windows, it is required to have the "exec" task with executable="sh"
and then pass the script name as an argument.
> This patch does just that.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message