diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/Cab.java b/src/main/org/apache/tools/ant/taskdefs/optional/Cab.java
index 78bf1da6d..8a0c1eebf 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/Cab.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/Cab.java
@@ -90,8 +90,6 @@ public class Cab extends MatchingTask {
     
     protected String archiveType = "cab";
 
-    private static String myos;
-
     private FileUtils fileUtils = FileUtils.newFileUtils();
 
     /**
@@ -288,8 +286,7 @@ public class Cab extends MatchingTask {
 
         log("Building "+ archiveType +": "+ cabFile.getAbsolutePath());
 
-        // we must be on Windows to continue
-        if (Os.isFamily("windows")) {
+        if (!Os.isFamily("windows")) {
             log("Using listcab/libcabinet", Project.MSG_VERBOSE);
             
             StringBuffer sb = new StringBuffer();