William Stearns <[EMAIL PROTECTED]> 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."
[EMAIL PROTECTED]
SuSE GmbH, Schanzäckerstr. 10, D-90443 Nürnberg
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to