[89217] trunk/base/src/port1.0/portutil.tcl

[jmr at macports.org]

Revision: 89217
          http://trac.macports.org/changeset/89217
Author:   jmr at macports.org
Date:     2012-01-21 04:14:51 -0800 (Sat, 21 Jan 2012)
Log Message:
-----------
add missing dry-run check

Modified Paths:
--------------
    trunk/base/src/port1.0/portutil.tcl

Modified: trunk/base/src/port1.0/portutil.tcl
===================================================================
--- trunk/base/src/port1.0/portutil.tcl	2012-01-21 11:42:11 UTC (rev 89216)
+++ trunk/base/src/port1.0/portutil.tcl	2012-01-21 12:14:51 UTC (rev 89217)
@@ -1594,17 +1594,18 @@
         }
     }
 
-    if {![file isdirectory $workpath] && ![tbool ports_dryrun]} {
-        file mkdir $workpath
-        chownAsRoot $subbuildpath
+    if {![tbool ports_dryrun]} {
+        if {![file isdirectory $workpath]} {
+            file mkdir $workpath
+            chownAsRoot $subbuildpath
+        }
+        # Create a symlink to the workpath for port authors
+        if {[tbool place_worksymlink] && ![file isdirectory $worksymlink]} {
+            ui_debug "Attempting ln -sf $workpath $worksymlink"
+            ln -sf $workpath $worksymlink
+        }
     }
 
-    # Create a symlink to the workpath for port authors
-    if {[tbool place_worksymlink] && ![file isdirectory $worksymlink]} {
-        ui_debug "Attempting ln -sf $workpath $worksymlink"
-        ln -sf $workpath $worksymlink
-    }
-
     # de-escalate privileges if MacPorts was started with sudo
     dropPrivileges
 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macports-changes/attachments/20120121/ebb26880/attachment-0001.html>