RFE: macports installer package postflight tweak

[asigel@…]

mktemp is used in the postflight script of the installer. the return of the mktemp is not checked so it may not have succeeded. the subsequent write to can fail when it doesnt have to.

Instead of doing this:

BINPATH=/opt/local/bin
SBINPATH=/opt/local/sbin
TMP=`/usr/bin/mktemp /tmp/dp.$$`
$SHELL -l <<EOF > $TMP
    /usr/bin/printenv PATH
    exit
EOF

if grep $BINPATH $TMP >/dev/null 2>&1; then

maybe you can do something like:

if $SHELL -c "/usr/bin/printenv PATH"|grep -c $BINPATH >/dev/null; then
        echo "You already have the right PATH - l337!"
else

most people would probably never run into a failure case.. but why create a temp file if you dont have to..