Changeset 78528 in webkit

Timestamp:

Feb 14, 2011 6:23:24 PM (13 years ago)

Author:

dpranke@chromium.org

Message:

2011-02-14 Dirk Pranke <​dpranke@chromium.org>

Unreviewed, build fix.

Fix minor regression introduced in r78522. For some reason, Python
provides a platform.mac_ver() routine on Linux, so we need to
check to make sure it's returning something in addition to
checking for its existence.

• Scripts/webkitpy/layout_tests/port/mac.py:

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/webkitpy/layout_tests/port/mac.py (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2011-02-14  Dirk Pranke  <dpranke@chromium.org>
+
+        Unreviewed, build fix.
+
+        Fix minor regression introduced in r78522. For some reason, Python
+        provides a platform.mac_ver() routine on Linux, so we need to
+        check to make sure it's returning something in addition to
+        checking for its existence.
+
+        * Scripts/webkitpy/layout_tests/port/mac.py:
+
 2011-02-14  Dirk Pranke  <dpranke@chromium.org>
 

trunk/Tools/Scripts/webkitpy/layout_tests/port/mac.py

@@ -42 +42 @@
     # We only support Tiger, Leopard, and Snow Leopard.
     if not os_version_string:
-        if hasattr(platform, 'mac_ver'):
+        if hasattr(platform, 'mac_ver') and platform.mac_ver()[0]:
             os_version_string = platform.mac_ver()[0]
         else: