[94077] trunk/dports/java/hadoop/Portfile

[ryandesign at macports.org]

Revision: 94077
          https://trac.macports.org/changeset/94077
Author:   ryandesign at macports.org
Date:     2012-06-08 11:57:32 -0700 (Fri, 08 Jun 2012)
Log Message:
-----------
hadoop: check OS version before fetching distfile; simplify how OS version is checked (using ${os.major})

Modified Paths:
--------------
    trunk/dports/java/hadoop/Portfile

Modified: trunk/dports/java/hadoop/Portfile
===================================================================
--- trunk/dports/java/hadoop/Portfile	2012-06-08 18:26:11 UTC (rev 94076)
+++ trunk/dports/java/hadoop/Portfile	2012-06-08 18:57:32 UTC (rev 94077)
@@ -24,6 +24,15 @@
 checksums           rmd160  a98ca552a8a6dfcc6455b53cfd7988d562c265d1 \
                     sha256  a33d07ece0e9b7e1ecf32670cac28444ba8f56b3360548d96b56964facc2d1ef
 
+pre-fetch {
+    # This port works on Mac OS X 10.6 or later, because 'sudo option -E'
+    # is not available on Mac OS X 10.5 or earlier. See #34665.
+    if {${os.platform} == "darwin" && ${os.major} <= 9} {
+        ui_error "This port works on Mac OS X 10.6 (Snow Leopard) or later"
+        return -code error "Mac OS X 10.6 (Snow Leopard) or later is required"
+    }
+}
+
 patchfiles          patch-hadoop-env.sh.diff
 
 use_configure       no
@@ -39,14 +48,6 @@
         ui_error "Java 1.6 is required, but not located at ${java_home}"
         return -code error "Java 1.6 missing"
     }
-
-    # This port works on Mac OS X 10.6 or later, because 'sudo option -E'
-    # is not available on Mac OS X 10.5 or earlier. See #34665.
-    set os_major_version  [lindex [split ${os.version} .] 0]
-    if {${os_major_version} <= 9} {
-        ui_error "This port works on Mac OS X 10.6 (Snow Leopard) or later"
-        return -code error "Mac OS X 10.6 (Snow Leopard) or later is required"
-    }
 }
 
 # Hadoop home and conf directories.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macports-changes/attachments/20120608/a1b879a7/attachment-0001.html>