Changeset 79689

Timestamp:

06/23/11 10:21:02 (4 years ago)

Author:

jmr@…

Message:

don't try to drop privileges if macportsuser does not exist

Location:

trunk/base/src

Files:

• macports1.0/macports.tcl (modified) (1 diff)
• port1.0/portutil.tcl (modified) (1 diff)

trunk/base/src/macports1.0/macports.tcl

@@ -1631 +1631 @@
     set workername [ditem_key $mport workername]
 
+    # check for existence of macportsuser and use fallback if necessary
+    $workername eval validate_macportsuser
     # check variants
     if {[$workername eval check_variants $target] != 0} {

trunk/base/src/port1.0/portutil.tcl

@@ -2508 +2508 @@
 }
 
+proc validate_macportsuser {} {
+    global macportsuser
+    if {[getuid] == 0 && $macportsuser != "root" && 
+        ([existsuser $macportsuser] == 0 || [existsgroup $macportsuser] == 0 )} {
+        ui_warn "configured user/group $macportsuser does not exist, will build as root"
+        set macportsuser "root"
+    }
+}
+
 # dependency analysis helpers