[90019] trunk/base/src/macports1.0/macports.tcl

jberry at macports.org jberry at macports.org
Sun Feb 19 12:16:05 PST 2012


Revision: 90019
          http://trac.macports.org/changeset/90019
Author:   jberry at macports.org
Date:     2012-02-19 12:16:05 -0800 (Sun, 19 Feb 2012)
Log Message:
-----------
Ask mdls for raw values to avoid having to parse its output

Modified Paths:
--------------
    trunk/base/src/macports1.0/macports.tcl

Modified: trunk/base/src/macports1.0/macports.tcl
===================================================================
--- trunk/base/src/macports1.0/macports.tcl	2012-02-19 20:08:13 UTC (rev 90018)
+++ trunk/base/src/macports1.0/macports.tcl	2012-02-19 20:16:05 UTC (rev 90019)
@@ -449,8 +449,8 @@
             ui_error
             ui_error "Please use xcode-select to select an Xcode installation:"
             foreach xcode $installed_xcodes {
-                regexp {kMDItemVersion = "([\d.]+)"} [exec $mdls -name kMDItemVersion $xcode] match vers
-                if {![info exists vers]} { set vers "unknown" }
+                set vers [exec $mdls -raw -name kMDItemVersion $xcode]
+                if { $vers == "(null)" } { set vers "unknown" }
                 if {[vercmp $vers 4.3] >= 0 || [_is_valid_developer_dir "${xcode}/Contents/Developer"]} {
                     ui_error "    sudo xcode-select -switch ${xcode} # version ${vers}"
                 } elseif {[_is_valid_developer_dir "${xcode}/../.."]} {
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macports-changes/attachments/20120219/722ac029/attachment.html>


More information about the macports-changes mailing list