Re: [Help-smalltalk] BSD sed fix

[Paolo Bonzini]

> Additionally, it appears the installed packages.xml file was omitted
> so only the files specified on the command line where merged and
> installed.  I assume the installed one is supposed to be implicit?
There is a bug in the code that tries to make it implicit *if it is not
specified on the command line*.  That's above the non-portable sed
invocation.  The correct snippet is:

  base="$DESTDIR$IMAGE_PATH/packages.xml"
  if test -f $base; then
    for i in $files; do
      if test $i -ef $base; then
        add=
        break
      fi
    done
    files="$base${base:+ }$files"
  fi

  echo "Merging $files..."
  $dry_run || {
    sed -e '$i\
</packages>' -e '/<.packages>/,/<packages>/d' \
      $files > packages.tmp

    mv packages.tmp $DESTDIR$IMAGE_PATH/packages.xml
  }

Paolo