[33926] trunk/base

jberry at macports.org jberry at macports.org
Thu Feb 7 10:23:24 PST 2008


Revision: 33926
          http://trac.macosforge.org/projects/macports/changeset/33926
Author:   jberry at macports.org
Date:     2008-02-07 10:22:03 -0800 (Thu, 07 Feb 2008)

Log Message:
-----------
Disable code that installs paths for MacPorts into /etc/manpathds.d and /etc/paths.d.

At the worst, this code wasn't perfect as MacPorts paths then come after system paths, and thus don't override system commands. At best, this code at least gets MANPATH set Leopard. Disabled pending further feedback.

Modified Paths:
--------------
    trunk/base/configure
    trunk/base/configure.ac

Modified: trunk/base/configure
===================================================================
--- trunk/base/configure	2008-02-07 17:32:38 UTC (rev 33925)
+++ trunk/base/configure	2008-02-07 18:22:03 UTC (rev 33926)
@@ -12173,11 +12173,13 @@
 
 
 # If /etc/paths.d and /etc/manpaths.d then we can install our path configuration
+# (this code is currently disabled since this mechanism implies that MacPorts paths are
+# added after the system's paths, and so MacPorts commands would not override system functions).
 { echo "$as_me:$LINENO: checking whether to install MacPorts paths into /etc/paths.d and /etc/manpaths.d" >&5
 echo $ECHO_N "checking whether to install MacPorts paths into /etc/paths.d and /etc/manpaths.d... $ECHO_C" >&6; }
 mpprefix=$prefix
 test "x$mpprefix" = "xNONE" && mpprefix=$ac_default_prefix
-if test -d /etc/paths.d -a -d /etc/manpaths.d -a "$mpprefix" = "$ac_default_prefix"; then
+if false && test -d /etc/paths.d -a -d /etc/manpaths.d -a "$mpprefix" = "$ac_default_prefix"; then
 	PATHCONF=pathconf
 	{ echo "$as_me:$LINENO: result: yes" >&5
 echo "${ECHO_T}yes" >&6; }

Modified: trunk/base/configure.ac
===================================================================
--- trunk/base/configure.ac	2008-02-07 17:32:38 UTC (rev 33925)
+++ trunk/base/configure.ac	2008-02-07 18:22:03 UTC (rev 33926)
@@ -325,10 +325,12 @@
 	
 	
 # If /etc/paths.d and /etc/manpaths.d then we can install our path configuration
+# (this code is currently disabled since this mechanism implies that MacPorts paths are
+# added after the system's paths, and so MacPorts commands would not override system functions).
 AC_MSG_CHECKING(whether to install MacPorts paths into /etc/paths.d and /etc/manpaths.d)
 mpprefix=$prefix
 test "x$mpprefix" = "xNONE" && mpprefix=$ac_default_prefix
-if test -d /etc/paths.d -a -d /etc/manpaths.d -a "$mpprefix" = "$ac_default_prefix"; then
+if false && test -d /etc/paths.d -a -d /etc/manpaths.d -a "$mpprefix" = "$ac_default_prefix"; then
 	PATHCONF=pathconf
 	AC_MSG_RESULT([yes])
 else

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.macosforge.org/pipermail/macports-changes/attachments/20080207/a23b01b9/attachment.html


More information about the macports-changes mailing list