Re: [ANNONCE] Kernel Autoconfiguration utility v.0.9.1.2

[Andreas Schwab <schwab@suse.de>]

William Stearns <wstearns@pobox.com> writes:

|> Good day, Giacomo,
|> 
|> On Thu, 15 Feb 2001, Giacomo Catenazzi wrote:
|> 
|> > How to use: (now, testing phase)
|> >   unpack the files (better: in a new directory)
|> >   > bash autoconfigure.sh | less
|> >   check the output.
|> >   no super user privileges required!
|> 
|> 	Nice work - that's a neat way to do it.
|> 	One detail; you appear to assume that bash2 is being used.  If
|> bash1 is /bin/bash, one gets syntax errors.  The following patch allows
|> the script to run under bash1 and bash2, with no noticeable problems.
|> 
|> --- autoconfigure.sh-0.9.1.2.orig	Wed Feb 14 15:37:30 2001
|> +++ autoconfigure.sh	Thu Feb 15 10:59:45 2001
|> @@ -109,7 +109,7 @@
|>  }
|>  function found () {
|>      local conf=CONFIG_$1
|> -    if [ "${!conf}" !=  "y" ]; then
|> +    if [ "${conf}" !=  "y" ]; then
|>  	define $1 y
|>      else
|>  	debug "$1=y"

This is plain wrong.  ${!conf} and ${conf} are completely different
things.

Andreas.

-- 
Andreas Schwab                                  "And now for something
SuSE Labs                                        completely different."
Andreas.Schwab@suse.de
SuSE GmbH, Schanzäckerstr. 10, D-90443 Nürnberg