This looks pretty reasonable to me. I'm sure "set -- fixincl.x"
meant something two decades ago when I wrote it. :)
I should update my e-address since I can be off gnu mail for
weeks at a time nowadays. (We retired folk are less consistent
about checking project emails ...)
Oh, the assert.h thingy seems fine, too. RE:
https://gcc.gnu.org/ml/gcc-patches/2018-06/msg00370.html
On 06/09/18 06:22, Rasmus Villemoes wrote:
> 2018-06-09 Rasmus Villemoes <[email protected]>
>
> fixincludes/
>
> * genfixes: exit 1 when autogen not found.
> * genfixes: Remove some redundant code.
> * genfixes: Update URL to autogen source code.
> ---
> fixincludes/genfixes | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/fixincludes/genfixes b/fixincludes/genfixes
> index f0fc5e64f8b..47aad01289d 100755
> --- a/fixincludes/genfixes
> +++ b/fixincludes/genfixes
> @@ -55,19 +55,16 @@ do
> esac
> done
>
> -if [ $# -eq 0 ] ; then
> - set -- fixincl.x
> -fi
> -
> AG="autogen $AG"
> set -e
>
> if [ -z "`${AG} -v | fgrep ' 5.'`" ]
> then
> echo "AutoGen appears to be out of date or not correctly installed."
> - echo "Please download and install:"
> - echo " ftp://gcc.gnu.org/pub/gcc/infrastructure/autogen.tar.gz"
> + echo "Please download and install from:"
> + echo " https://ftp.gnu.org/gnu/autogen/"
> touch fixincl.x
> + exit 1
> else
> echo AutoGen-ing fixincl.x
> $AG inclhack.def
>