Re: [PATCH] bootstrap: avoid broken Solaris sed

[Bruno Haible]

Eric Blake wrote:
> -package=`sed -n "$extract_package_name" configure.ac` || exit
> +package=`$SED -n "$extract_package_name" configure.ac` || exit

Instead of changing every use of sed to $SED, why don't you define 'sed'
as a function? Like this:

  if test "$SED" != sed; then
    sed ()
    {
      $SED "$@"
    }
  fi

This will
  1. follow the gnulib philosophy of separation of platform specific
     workarounds and the actual code that uses the workarounds,
  2. reduce the danger that someone - not aware of of the limitations of sed -
     uses 'sed' instead of '$SED' in future modifications of the script.

Bruno