[32628] branches/release_1_6/base/portmgr/dmg/postflight

[jmpp at macports.org]

Revision: 32628
          http://trac.macosforge.org/projects/macports/changeset/32628
Author:   jmpp at macports.org
Date:     2008-01-09 21:50:37 -0800 (Wed, 09 Jan 2008)

Log Message:
-----------

 * echo's from this script go to a system log (/var/log/installer to be precise), so \n characters to aid in readability make no sense, strip them;
 * Add informational messages to the log about particular settings having been added;

Modified Paths:
--------------
    branches/release_1_6/base/portmgr/dmg/postflight

Modified: branches/release_1_6/base/portmgr/dmg/postflight
===================================================================
--- branches/release_1_6/base/portmgr/dmg/postflight	2008-01-10 05:37:45 UTC (rev 32627)
+++ branches/release_1_6/base/portmgr/dmg/postflight	2008-01-10 05:50:37 UTC (rev 32628)
@@ -89,7 +89,7 @@
 # Command to update the MacPorts installation through "selfupdate":
 function update_macports {
     export PATH=$BINPATH:$PATH
-    echo -e "\nSynchronizing the MacPorts installation with the project's rsync server...\n"
+    echo "Synchronizing the MacPorts installation with the project's rsync server..."
     port -v selfupdate || {
         echo "An attempt to synchronize your recent MacPorts installation with the project's rsync server failed!"
         echo "Please run 'port -d selfupdate' manually to find out the cause of the error."
@@ -116,13 +116,14 @@
 esac
 
 
-echo -e "\nChecking the shell environment for user \"$USER\"...\n"
+echo "Checking the shell environment for user \"$USER\"..."
 
 # Adding our setting to the PATH variable if not already there:
 if $SHELL $LOGIN_FLAG -c "/usr/bin/printenv PATH" | grep $PREFIX > /dev/null 2>&1; then
     echo "Your shell already has the right PATH environment variable for use with MacPorts!"
 else
     write_setting PATH
+    echo "An appropriate PATH variable has been added to your environment by the MacPorts installed."
 fi
 
 # We gather the path into a variable of our own for faster operation:
@@ -142,6 +143,7 @@
     echo "Your shell already has the right MANPATH environment variable for use with MacPorts!"
 else
     write_setting MANPATH
+    echo "An appropriate MANPATH variable has been added to your environment by the MacPorts installed."
 fi
 
 # Adding a DISPLAY variable only if we're running on Tiger or less and if it doesn't already exist:
@@ -149,11 +151,12 @@
     echo "Your shell already has the right DISPLAY environment variable for use with MacPorts!"
 else
     write_setting DISPLAY
+    echo "An appropriate DISPLAY variable has been added to your environment by the MacPorts installed."
 fi
 
 
-# Postflight script is done with its job!
+# Postflight script is done with its job, update MacPorts and exit gracefully!
 update_macports
-echo -e "\nYou have succesfully installed the MacPorts system, launch a terminal and try it out!\n"
-echo -e "Read the port(1) manual page and http://guide.macports.org for help.\n"
+echo "You have succesfully installed the MacPorts system, launch a terminal and try it out!"
+echo "Read the port(1) manual page and http://guide.macports.org for help."
 exit 0

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