Add Peter's portability and option suggestions.
authorBruce Momjian <bruce@momjian.us>
Tue, 15 Jan 2002 04:05:24 +0000 (04:05 +0000)
committerBruce Momjian <bruce@momjian.us>
Tue, 15 Jan 2002 04:05:24 +0000 (04:05 +0000)
contrib/pg_upgrade/README
contrib/pg_upgrade/pg_upgrade
contrib/pg_upgrade/pg_upgrade.1 [moved from contrib/pg_upgrade/pg_upgrade.man with 100% similarity]

index f5fb65e7d4f5fd9fc18a11fd816fab0bdf42e647..46bff0ca4d172971ff777eb4a16af78463673dcc 100644 (file)
+                                 pg_upgrade
 
+This is a version of pg_upgrade which will migrate a 7.1 database to 7.2,
+or allow a 7.2 to 7.2 migration if you need to perform an initdb.
 
+Read the manual page for more information.  To view it:
 
-PG_UPGRADE(1)     PostgreSQL Client Applications    PG_UPGRADE(1)
+       nroff -man pg_upgrade.1 | $PAGER
 
+Bruce Momjian
 
-N\bNA\bAM\bME\bE
-       pg_upgrade - upgrade tool
+2002-01-14
 
-S\bSY\bYN\bNO\bOP\bPS\bSI\bIS\bS
-       Allows upgrading from a previous release without reloading
-       data
+--
 
-D\bDE\bES\bSC\bCR\bRI\bIP\bPT\bTI\bIO\bON\bN
-       p\bpg\bg_\b_u\bup\bpg\bgr\bra\bad\bde\be is a utility  for  upgrading  from  a  previous
-       PostgreSQL release without reloading all the data. Not all
-       PostgreSQL releases  can  use  this  utility.   Check  the
-       release notes for details about your version.
-
-       p\bpg\bg_\b_u\bup\bpg\bgr\bra\bad\bde\be  must  be  run in two stages.  In phase one you
-       must run p\bpg\bg_\b_u\bup\bpg\bgr\bra\bad\bde\be with your old database installation in
-       place.   In phase two, p\bpg\bg_\b_u\bup\bpg\bgr\bra\bad\bde\be must be run on a freshly
-       i\bin\bni\bit\btd\bdb\bb'ed current  install.   In  both  phases,  the  same
-       p\bpg\bg_\b_u\bup\bpg\bgr\bra\bad\bde\be  script  matching  the  newly installed version
-       must be used.
-
-U\bUp\bpg\bgr\bra\bad\bdi\bin\bng\bg P\bPo\bos\bst\btg\bgr\bre\beS\bSQ\bQL\bL w\bwi\bit\bth\bh p\bpg\bg_\b_u\bup\bpg\bgr\bra\bad\bde\be
-       1) Back up your existing  data  directory,  preferably  by
-       making a complete dump with p\bpg\bg_\b_d\bdu\bum\bmp\bpa\bal\bll\bl.\b.
-
-       2)  Copy  the  program _\bp_\bg_\bs_\bq_\bl_\b/_\bc_\bo_\bn_\bt_\br_\bi_\bb_\b/_\bp_\bg_\b__\bu_\bp_\bg_\br_\ba_\bd_\be_\b/_\bp_\bg_\b__\bu_\bp_\bg_\br_\ba_\bd_\be
-       from the current PostgreSQL  distribution  somewhere  into
-       your path.
-
-       3)  Change your working directory to the pgsql main direc-
-       tory, and type:
-
-       $\b$ p\bpg\bg_\b_u\bup\bpg\bgr\bra\bad\bde\be -\b-1\b1
-
-       to collect information about the old database  needed  for
-       the upgrade.
-
-       4) Do:
-
-       $\b$ m\bma\bak\bke\be i\bin\bns\bst\bta\bal\bll\bl
-
-       to install the new binaries.
-
-       5)  Do:
-
-       $\b$ c\bcd\bd p\bpg\bgs\bsq\bql\bl/\b/c\bco\bon\bnt\btr\bri\bib\bb/\b/p\bpg\bg_\b_r\bre\bes\bse\bet\btx\bxl\blo\bog\bg
-       $\b$ m\bma\bak\bke\be i\bin\bns\bst\bta\bal\bll\bl
-
-       to  install  the  _\bp_\bg_\b__\br_\be_\bs_\be_\bt_\bx_\bl_\bo_\bg  utility which is needed by
-       p\bpg\bg_\b_u\bup\bpg\bgr\bra\bad\bde\be.
-
-       6) Run initdb to create a new template1 database  contain-
-       ing  the system tables for the new release.  Make sure you
-       use settings similar to those used in your  previous  ver-
-       sion.
-
-
-
-14 Jan 2002               PG_UPGRADE(1)                         1
-
-
-
-
-
-PG_UPGRADE(1)     PostgreSQL Client Applications    PG_UPGRADE(1)
-
-
-       7)  Start  the new _\bp_\bo_\bs_\bt_\bm_\ba_\bs_\bt_\be_\br_\b.  (Note: it is critical that
-       no users connect to the server until the upgrade  is  com-
-       plete.  You may wish to start the postmaster without -i or
-       alter pg_hba.conf temporarily.)
-
-       8)  Change your working directory to the pgsql main direc-
-       tory again, and type:
-
-       $\b$ p\bpg\bg_\b_u\bup\bpg\bgr\bra\bad\bde\be -\b-2\b2
-
-       The  program will do some checking to make sure everything
-       is properly configured, and will  then  recreate  all  the
-       databases  and  tables you had, but with no data.  It will
-       then physically move the data files containing  non-system
-       tables and indexes into the proper subdirectories.
-
-       9)   Restore  your old _\bp_\bg_\b__\bh_\bb_\ba_\b._\bc_\bo_\bn_\bf if needed to allow user
-       logins.
-
-
-       10)   Carefully  examine  the  contents  of  the  upgraded
-       databases.  If you detect problems, you'll need to recover
-       by restoring from your full  p\bpg\bg_\b_d\bdu\bum\bmp\bpa\bal\bll\bl  backup.  You  can
-       delete  the _\bp_\bg_\b__\bu_\bp_\bg_\br_\ba_\bd_\be_\b__\bi_\bn_\bf_\bo_\b/ directory when you are satis-
-       fied.
-
-       The upgraded databases will be in  an  un-vacuumed  state.
-       You  will  probably  want  to  run a _\bV_\bA_\bC_\bU_\bU_\bM _\bA_\bN_\bA_\bL_\bY_\bZ_\bE before
-       beginning production work.
-
-S\bSE\bEE\bE A\bAL\bLS\bSO\bO
-       initdb(1), postmaster(1), pg_dump(1), pg_dumpall(1), vacu-
-       umdb(1)
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-14 Jan 2002               PG_UPGRADE(1)                         2
+To migrate this to newer versions of PostgreSQL:  
 
+       1)  Update the version numbers at the top of the file
+       2)  Search for specific version mentions in the script and update
+           accordingly.
+       3)  Add changes for next version.
 
index 939bf0ed80d57ee5148d64a9bcca1b97ff78aa39..44d31ab59d33c7184742575348ddebb65b0f3181 100755 (executable)
@@ -3,7 +3,7 @@
 # pg_upgrade: update a database without needing a full dump/reload cycle.
 # CAUTION: Read the manual page before trying to use this!
 
-# $Header: /cvsroot/pgsql/contrib/pg_upgrade/Attic/pg_upgrade,v 1.2 2002/01/14 22:54:44 momjian Exp $
+# $Header: /cvsroot/pgsql/contrib/pg_upgrade/Attic/pg_upgrade,v 1.3 2002/01/15 04:05:24 momjian Exp $
 #
 # NOTE: we must be sure to update the version-checking code a few dozen lines
 # below for each new PostgreSQL release.
@@ -11,7 +11,7 @@
 #set -x
 
 # Set this to "Y" to enable this program
-ENABLE="N"
+ENABLE="Y"
 
 if [ "$ENABLE" != "Y" ]
 then
@@ -29,6 +29,7 @@ CUR_VERSION="7.2"
 
 trap "rm -f /tmp/$$.*" 0 1 2 3 15
 
+BASENAME=`basename "$0`
 PHASE=""
 
 if [ "$#" -eq 1 ]
@@ -40,40 +41,42 @@ then
        then    PHASE="2"
                shift
        elif [ "X$1" = "X-D" ]
-       then    DATADIR="$2"
-               shift
+       then    PGDATA="$2"
+               shift 2
        fi
 fi
 
 if [ "$PHASE" = "" ]
-then   echo "You must run $0 in either mode 1 or mode 2." 1>&2
-       echo "Usage:  $0 [-D datadir] -1 | -2" 1>&2
+then   echo "You must run $BASENAME in either mode 1 or mode 2." 1>&2
+       echo "Usage:  $BASENAME [-D datadir] -1 | -2" 1>&2
        exit 1
 fi
 
-if [ "$DATADIR" = "" ]
-then   echo "You must set the DATADIR environment variable or specify it with -D." 1>&2
-       echo "Usage:  $0 [-D datadir] -1 | -2" 1>&2
+if [ "$PGDATA" = "" ]
+then   echo "You must set the PGDATA environment variable or specify it with -D." 1>&2
+       echo "Usage:  $BASENAME [-D datadir] -1 | -2" 1>&2
        exit 1
 fi
 
-if [ ! -d "$DATADIR" ]
-then   echo "$DATADIR does not exist.
-$0 aborted." 1>&2
+if [ ! -d "$PGDATA" ]
+then   echo "$PGDATA does not exist.
+$BASENAME aborted." 1>&2
        if [ "$PHASE" -eq 2 ]
        then    echo "Perhaps you didn't run initdb." 1>&2
        fi
        exit 1
 fi
 
-if [ "$USER" = "root" -o ! -r "$DATADIR"/PG_VERSION ]
+if [ "$USER" = "root" -o ! -r "$PGDATA"/PG_VERSION ]
 then   echo "You must run this as the PostgreSQL superuser.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
        exit 1
 fi
 
-INFODIR="pg_upgrade_info"
-SAVEDATA="$INFODIR"/"`basename \"$DATADIR\"`"
+# Strip off the trailing directory name and store our data there
+# in the hope we are in the same filesystem so 'mv 'works.
+INFODIR=`dirname "$PGDATA"`/pg_upgrade_info
+SAVEDATA="$INFODIR"/data
 
 make_dbobjoidmap()
 {
@@ -119,35 +122,35 @@ move_objfiles()
             ! -h "$SAVEDATA"/base/"$SRC_DBOID"/"$SRC_OID" ]
        then    echo "Move of database $DB, OID $SRC_OID, object $OBJ failed.
 File not found;  exiting" 1>&2
-               exit 1
+               return 1
        fi
 
-       if [ ! -f "$DATADIR"/base/"$DST_DBOID"/"$DST_OID" -a \
-            ! -h "$DATADIR"/base/"$DST_DBOID"/"$DST_OID" ]
+       if [ ! -f "$PGDATA"/base/"$DST_DBOID"/"$DST_OID" -a \
+            ! -h "$PGDATA"/base/"$DST_DBOID"/"$DST_OID" ]
        then    echo "Move of database $DB, OID $DST_OID, object $OBJ failed.
 File not found;  exiting" 1>&2
-               exit 1
+               return 1
        fi
 
        # Move files
 
-       mv -f "$SAVEDATA"/base/"$SRC_DBOID"/"$SRC_OID" "$DATADIR"/base/"$DST_DBOID"/"$DST_OID"
+       mv -f "$SAVEDATA"/base/"$SRC_DBOID"/"$SRC_OID" "$PGDATA"/base/"$DST_DBOID"/"$DST_OID"
        if [ "$?" -ne 0 ]
        then    echo "Move of database $DB, OID $SRC_OID, object $OBJ
 to $DST_OID failed.;  exiting" 1>&2
-               exit 1
+               return 1
        fi
 
        # handle table extents
 
-       ls "$SAVEDATA"/base/"$SRC_DBOID"/"$SRC_OID".* 2>/dev/null | while read FILE
+       ls "$SAVEDATA"/base/"$SRC_DBOID"/"$SRC_OID".* 2> /dev/null | while read FILE
        do
                EXT=`basename "$FILE" | sed 's/^.*\.\(.*\)$/\1/'`
-               mv -f "$FILE" "$DATADIR"/base/"$DST_DBOID"/"$DST_OID"."$EXT"
+               mv -f "$FILE" "$PGDATA"/base/"$DST_DBOID"/"$DST_OID"."$EXT"
                if [ "$?" -ne 0 ]
                then    echo "Move of database $DB, OID $SRC_OID, object $OBJ
 to $DST_OID failed.;  exiting" 1>&2
-                       exit 1
+                       return 1
                fi
        done
 }
@@ -160,44 +163,56 @@ then
        ##########################
 
 
-       if [ ! -d "$DATADIR"/base/1 ]
-       then    echo "There is no database template1 in $DATADIR/base." 1>&2
+       if [ ! -d "$PGDATA"/base/1 ]
+       then    echo "There is no database template1 in $PGDATA/base." 1>&2
                exit 1
        fi
 
        # get version
-       SRC_VERSION=`cat "$DATADIR"/PG_VERSION`
+       SRC_VERSION=`cat "$PGDATA"/PG_VERSION`
        if [ "$SRC_VERSION" = "" ]
-       then    echo "$0 can not find PostgreSQL version file '$DATADIR/PG_VERSION'.
-$0 aborted." 1>&2
+       then    echo "$BASENAME can not find PostgreSQL version file '$PGDATA/PG_VERSION'.
+$BASENAME aborted." 1>&2
                exit 1
        fi
 
        if [    "$SRC_VERSION" != "$CUR_VERSION" -a \
                "$SRC_VERSION" != "$UPGRADE_VERSION" ]
-       then    echo 
-       "$0 supports versions $UPGRADE_VERSION and $CUR_VERSION only." 1>&2
-               echo 
-       "However, your database is version $SRC_VERSION;
-$0 aborted." 1>&2
+       then    echo "$BASENAME supports versions $UPGRADE_VERSION and $CUR_VERSION only." 1>&2
+               echo "However, your database is version $SRC_VERSION;
+$BASENAME aborted." 1>&2
                echo "You will need to dump and restore using pg_dumpall." 1>&2
                exit 1
        fi
 
        # Start server, if needed, so we can do some work.
-       if ! pg_ctl status | head -1 | grep -q "is running"
+       if ! pg_ctl status | sed -n '1p' | grep "is running" > /dev/null 2>&1
        then    pg_ctl -w start
                if [ $? -ne 0 ]
                then    echo "Can not start server.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
                        exit 1
                fi
        fi
 
        # create directory for our data
-       rm -rf "$INFODIR"
-       mkdir "$INFODIR"
-       chmod og-rwx "$INFODIR"
+       if ! rm -rf "$INFODIR" 
+       then    echo "Deletion of old pg_upgrade_info directory failed, $INFODIR.
+$BASENAME aborted." 1>&2
+               exit 1
+       fi
+       if ! mkdir "$INFODIR"
+       then    echo "Creation of new pg_upgrade_info directory failed, $INFODIR.
+$BASENAME aborted." 1>&2
+               exit 1
+       fi
+
+       if ! chmod og-rwx "$INFODIR"
+       then    echo "Permission change on new pg_upgrade_info directory failed, $INFODIR.
+$BASENAME aborted." 1>&2
+               exit 1
+       fi
+
 
        # Dump schema
        pg_dumpall -s |
@@ -224,13 +239,13 @@ $0 aborted." 1>&2
        }' > "$INFODIR"/schema
        if [ $? -ne 0 ]
        then    echo "Can not dump schema.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
                exit 1
        fi
        
        # Generate mappings for database
-       make_dboidmap > "$INFODIR"/dboidmap
-       make_dbobjoidmap > "$INFODIR"/dbobjoidmap
+       make_dboidmap > "$INFODIR"/dboidmap || exit "$?"
+       make_dbobjoidmap > "$INFODIR"/dbobjoidmap || exit "$?"
 
        # Generate setval() script for 7.1 because it has int4 sequences
        # Sequence XIDs changed from 7.2beta4 to beta5;  we have to recreate them.
@@ -264,7 +279,7 @@ $0 aborted." 1>&2
        vacuumdb -a
        if [ $? -ne 0 ]
        then    echo "Can not vacuum server.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
                exit 1
        fi
 
@@ -272,20 +287,21 @@ $0 aborted." 1>&2
        pg_ctl -w stop
        if [ $? -ne 0 ]
        then    echo "Can not stop server.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
                exit 1
        fi
-
-       mv "$DATADIR" "$INFODIR"
+       
+       # No matter what the directory name, call it data
+       mv "$PGDATA" "$INFODIR"/data
        if [ $? -ne 0 ]
-       then    echo "Can not move old /$DATADIR out of the way.
-$0 aborted." 1>&2
+       then    echo "Can not move old /$PGDATA out of the way.
+$BASENAME aborted." 1>&2
                exit 1
        fi
        echo
        echo
        echo "Plase 1 completed.
-Continue with the steps outlined in the $0 manual page."
+Continue with the steps outlined in the $BASENAME manual page."
        exit 0
 fi
 
@@ -297,12 +313,12 @@ fi
 # check things
 
 if [ ! -d "$INFODIR" ]
-then   echo "There is no '$INFODIR' directory from a phase 1 run of $0." 1>&2
+then   echo "There is no '$INFODIR' directory from a phase 1 run of $BASENAME." 1>&2
        exit 1  
 fi
 
 if [ ! -d "$SAVEDATA" ]
-then   echo "There is no '$SAVEDATA' directory from the phase 1 run of $0." 1>&2
+then   echo "There is no '$SAVEDATA' directory from the phase 1 run of $BASENAME." 1>&2
        exit 1  
 fi
 
@@ -311,46 +327,46 @@ then      echo "Cannot read '$SAVEDATA/PG_VERSION' --- something is wrong." 1>&2
        exit 1
 fi
 
-if [ ! -f "$DATADIR/PG_VERSION" ]
-then   echo "Cannot read '$DATADIR/PG_VERSION' --- something is wrong." 1>&2
+if [ ! -f "$PGDATA/PG_VERSION" ]
+then   echo "Cannot read '$PGDATA/PG_VERSION' --- something is wrong." 1>&2
        exit 1
 fi
 
-if [ ! -d "$DATADIR/base/1" ]
-then   echo "Cannot find database template1 in '$DATADIR/base'." 1>&2
-       echo "Are you running $0 as the postgres superuser?" 1>&2
+if [ ! -d "$PGDATA/base/1" ]
+then   echo "Cannot find database template1 in '$PGDATA/base'." 1>&2
+       echo "Are you running $BASENAME as the postgres superuser?" 1>&2
        exit 1
 fi
 
 # Get the actual versions seen in the data dirs.
 
 SRC_VERSION=`cat "$SAVEDATA"/PG_VERSION`
-DST_VERSION=`cat "$DATADIR"/PG_VERSION`
+DST_VERSION=`cat "$PGDATA"/PG_VERSION`
 
 # Check for version compatibility.
 # This code will need to be updated/reviewed for each new PostgreSQL release.
 
 if [ "$DST_VERSION" != "$CUR_VERSION" ]
-then   echo "$0 is for PostgreSQL version $CUR_VERSION
-but $DATADIR/PG_VERSION contains $DST_VERSION." 1>&2
+then   echo "$BASENAME is for PostgreSQL version $CUR_VERSION
+but $PGDATA/PG_VERSION contains $DST_VERSION." 1>&2
        echo "Did you run initdb for version $UPGRADE_VERSION by mistake?" 1>&2
        exit 1
 fi
 
 # Stop server for pg_resetxlog use
 
-if pg_ctl status | head -1 | grep -q "is running"
+if pg_ctl status | sed -n '1p' | grep "is running" > /dev/null 2>&1
 then   pg_ctl -w stop
        if [ $? -ne 0 ]
        then    echo "Can no start server.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
                exit 1
        fi
 fi
 
 # check for proper pg_resetxlog version
 
-pg_resetxlog 2>/dev/null
+pg_resetxlog 2> /dev/null
 # file not found status is normally 127, not 1
 if [ "$?" -ne 1 ]
 then   echo "Unable to find pg_resetxlog in path.
@@ -358,7 +374,7 @@ Install it from pgsql/contrib/pg_resetxlog and continue.;  exiting" 1>&2
        exit 1
 fi
 
-if ! pg_resetxlog -x 2>&1 | grep -q 'xid'
+if ! pg_resetxlog -x 2>&1 | grep 'xid' > /dev/null 2>&1
 then   echo "Old version of pg_resetxlog found in path.
 Install a newer version from pgsql/contrib/pg_resetxlog.;  exiting" 1>&2
        exit 1
@@ -369,16 +385,16 @@ fi
 
 SRC_XID=`pg_resetxlog -n "$SAVEDATA" | grep "NextXID" | awk -F'  *' '{print $4}'`
 if [ "$SRC_VERSION" = "7.1" -a "$SRC_XID" -gt 2000000000 ]
-then   echo "XID too high for $0.;  exiting" 1>&2
+then   echo "XID too high for $BASENAME.;  exiting" 1>&2
        exit 1
 fi
-DST_XID=`pg_resetxlog -n "$DATADIR" | grep "NextXID" | awk -F'  *' '{print $4}'`
+DST_XID=`pg_resetxlog -n "$PGDATA" | grep "NextXID" | awk -F'  *' '{print $4}'`
 
 # compare locales to make sure they match
 
 pg_resetxlog -n "$SAVEDATA" | grep "^LC_" > /tmp/$$.0
-pg_resetxlog -n "$DATADIR" | grep "^LC_" > /tmp/$$.1
-if ! diff /tmp/$$.0 /tmp/$$.1 >/dev/null
+pg_resetxlog -n "$PGDATA" | grep "^LC_" > /tmp/$$.1
+if ! diff /tmp/$$.0 /tmp/$$.1 > /dev/null
 then   echo "Locales do not match between the two versions.;  exiting" 1>&2
        exit 1
 fi
@@ -388,7 +404,7 @@ fi
 pg_ctl -w start
 if [ $? -ne 0 ]
 then   echo "Can not start server.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
        exit 1
 fi
 
@@ -403,7 +419,7 @@ fi
 psql template1 < "$INFODIR"/schema
 if [ $? -ne 0 ]
 then   echo "There were errors in the input script.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
        exit 1
 fi
 
@@ -416,13 +432,13 @@ echo "Input script complete, fixing row commit statuses..."
 vacuumdb -a
 if [ $? -ne 0 ]
 then   echo "There were errors during VACUUM.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
        exit 1
 fi
 
 # Generate mappings for database
-make_dboidmap > /tmp/$$.dboidmap
-make_dbobjoidmap > /tmp/$$.dbobjoidmap
+make_dboidmap > /tmp/$$.dboidmap || exit "$?"
+make_dbobjoidmap > /tmp/$$.dbobjoidmap || exit "$?"
 
 # we are done with SQL database access
 # shutdown forces buffers to disk
@@ -445,8 +461,8 @@ do
        # Skip system tables, except for pg_largeobject
        # pg_toast tables are handled later as part of the
        # base table move
-       if [ `expr "$OBJ" : 'pg_'` -eq 3 -a \
-            `expr "$OBJ" : 'pg_largeobject'` -ne 14 ]
+       if [ `expr X"$OBJ" : X'pg_'` -eq 4 -a \
+            `expr X"$OBJ" : X'pg_largeobject'` -ne 15 ]
        then    continue
        fi
 
@@ -458,7 +474,8 @@ do
        move_objfiles
 
        # Handle TOAST files if they exist
-       if grep -q "^$DB        pg_toast_$SRC_OID       " "$INFODIR"/dbobjoidmap
+       if grep "^$DB   pg_toast_$SRC_OID       " "$INFODIR"/dbobjoidmap \
+               > /dev/null 2>&1
        then    # toast heap
                SAVE_SRC_OID="$SRC_OID"
                SAVE_DST_OID="$DST_OID"
@@ -488,7 +505,7 @@ then        MAX_XID="$SRC_XID"
 else   MAX_XID="$DST_XID"
 fi
 
-pg_resetxlog -x "$MAX_XID" "$DATADIR"
+pg_resetxlog -x "$MAX_XID" "$PGDATA"
 if [ "$?" -ne 0 ]
 then   echo "Unable to set new XID.;  exiting" 1>&2
        exit 1
@@ -496,8 +513,8 @@ fi
 
 # Move over old WAL
 
-rm -r "$DATADIR"/pg_xlog
-mv -f "$SAVEDATA"/pg_xlog "$DATADIR"
+rm -r "$PGDATA"/pg_xlog
+mv -f "$SAVEDATA"/pg_xlog "$PGDATA"
 
 # Set last log file id and segment from old database
 
@@ -516,7 +533,7 @@ fi
 
 # Set checkpoint location of new database
 
-pg_resetxlog -l "$LOG_ID" "$SEG_ID" "$DATADIR"
+pg_resetxlog -l "$LOG_ID" "$SEG_ID" "$PGDATA"
 if [ "$?" -ne 0 ]
 then   echo "Unable to set new log file/segment id.;  exiting" 1>&2
        exit 1
@@ -538,13 +555,13 @@ then      echo "Set sequence values..."
        psql -d template1 -At < "$INFODIR"/setval
        if [ $? -ne 0 ]
        then    echo "There were errors during int4 sequence restore.
-$0 aborted." 1>&2
+$BASENAME aborted." 1>&2
                exit 1
        fi
 fi
 
 echo
 echo
-echo "You may remove the old database files with 'rm -r pg_upgrade'."
+echo "You may remove the old database files with 'rm -r $INFODIR'."
 
 exit 0