You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by dv...@apache.org on 2012/10/17 02:24:07 UTC

svn commit: r1399055 - in /pig/branches/branch-0.11: CHANGES.txt test/org/apache/pig/test/TestParamSubPreproc.java test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java

Author: dvryaboy
Date: Wed Oct 17 00:24:07 2012
New Revision: 1399055

URL: http://svn.apache.org/viewvc?rev=1399055&view=rev
Log:
PIG-2943: DevTests, Refactor Windows checks to use new Util.WINDOWS method for code health

Modified:
    pig/branches/branch-0.11/CHANGES.txt
    pig/branches/branch-0.11/test/org/apache/pig/test/TestParamSubPreproc.java
    pig/branches/branch-0.11/test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java

Modified: pig/branches/branch-0.11/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.11/CHANGES.txt?rev=1399055&r1=1399054&r2=1399055&view=diff
==============================================================================
--- pig/branches/branch-0.11/CHANGES.txt (original)
+++ pig/branches/branch-0.11/CHANGES.txt Wed Oct 17 00:24:07 2012
@@ -25,6 +25,8 @@ PIG-1891 Enable StoreFunc to make intell
 
 IMPROVEMENTS
 
+PIG-2943: DevTests, Refactor Windows checks to use new Util.WINDOWS method for code health (jgordon via dvryaboy)
+
 PIG-2794: Pig test: add utils to simplify testing on Windows (jgordon via gates)
 
 PIG-2908: Fix unit tests to work with jdk7 (rohini via dvryaboy)

Modified: pig/branches/branch-0.11/test/org/apache/pig/test/TestParamSubPreproc.java
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.11/test/org/apache/pig/test/TestParamSubPreproc.java?rev=1399055&r1=1399054&r2=1399055&view=diff
==============================================================================
--- pig/branches/branch-0.11/test/org/apache/pig/test/TestParamSubPreproc.java (original)
+++ pig/branches/branch-0.11/test/org/apache/pig/test/TestParamSubPreproc.java Wed Oct 17 00:24:07 2012
@@ -1198,7 +1198,7 @@ public class TestParamSubPreproc extends
             pigOStream = new FileWriter(basedir + "/output1.pig");
 
             String[] arg = {"date=`perl -e 'print \"20080228\n20070101\"' | head -n 1`"};
-            if (System.getProperty("os.name").toUpperCase().startsWith("WINDOWS"))
+            if (Util.WINDOWS)
                 arg[0] = "date=`perl -e 'print \\\"20080228\n20070101\\\"' | head -n 1`";
             String[] argFiles = null;
             ps.genSubstitutedFile(pigIStream , pigOStream , arg , argFiles);

Modified: pig/branches/branch-0.11/test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.11/test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java?rev=1399055&r1=1399054&r2=1399055&view=diff
==============================================================================
--- pig/branches/branch-0.11/test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java (original)
+++ pig/branches/branch-0.11/test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java Wed Oct 17 00:24:07 2012
@@ -144,7 +144,7 @@ public class TestTypeCheckingValidatorNe
 
     private static final String simpleEchoStreamingCommand;
     static {
-        if (System.getProperty("os.name").toUpperCase().startsWith("WINDOWS"))
+        if (Util.WINDOWS)
             simpleEchoStreamingCommand = "perl -ne 'print \\\"$_\\\"'";
         else
             simpleEchoStreamingCommand = "perl -ne 'print \"$_\"'";