2002-10-02 12:01:19

by ken hu

[permalink] [raw]
Subject: a problem report

Menuconfig has encountered a possible error in one of
the kernel's
configuration files and is unable to continue. Here
is the error
report:

Q> ./scripts/Menuconfig: line 823: MCmenu74: command
not found

Please report this to the maintainer <[email protected]>.
You may also
send a problem report to
<[email protected]>.

Please indicate the kernel version you are trying to
configure and
which menu you were trying to enter when this error
occurred.

make: *** [menuconfig] Error 1

Advanced Linux Sound Architecture menu i was trying
to enter when this error occurred.
kernel version is 2.5.40

__________________________________________________
Do you Yahoo!?
New DSL Internet Access from SBC & Yahoo!
http://sbc.yahoo.com


2002-10-02 12:46:05

by Sam Ravnborg

[permalink] [raw]
Subject: Re: a problem report

On Wed, Oct 02, 2002 at 05:06:47AM -0700, ken hu wrote:
> Menuconfig has encountered a possible error in one of
> the kernel's
Thanks, known problem.

Try this patch:
copy-n-pated so may not apply cleanly, try by hand.

Sam

--- linux/sound/Config.in 2002-10-01 12:09:44.000000000 +0200
+++ linux/sound/Config.in 2002-10-01 12:21:05.000000000 +0200
@@ -31,10 +31,7 @@
if [ "$CONFIG_SND" != "n" -a "$CONFIG_ARM" = "y" ]; then
source sound/arm/Config.in
fi
-if [ "$CONFIG_SND" != "n" -a "$CONFIG_SPARC32" = "y" ]; then
- source sound/sparc/Config.in
-fi
-if [ "$CONFIG_SND" != "n" -a "$CONFIG_SPARC64" = "y" ]; then
+if [ "$CONFIG_SND" != "n" -a "$CONFIG_SPARC32" = "y" ] || [ "$CONFIG_SND" !=
"n" -a "$CONFIG_SPARC64" = "y" ] ; then
source sound/sparc/Config.in
fi

2002-10-02 18:44:43

by Peter Samuelson

[permalink] [raw]
Subject: Re: a problem report

[Sam Ravnborg]
> --- linux/sound/Config.in 2002-10-01 12:09:44.000000000 +0200
> +++ linux/sound/Config.in 2002-10-01 12:21:05.000000000 +0200
> @@ -31,10 +31,7 @@
> if [ "$CONFIG_SND" != "n" -a "$CONFIG_ARM" = "y" ]; then
> source sound/arm/Config.in
> fi
> -if [ "$CONFIG_SND" != "n" -a "$CONFIG_SPARC32" = "y" ]; then
> - source sound/sparc/Config.in
> -fi
> -if [ "$CONFIG_SND" != "n" -a "$CONFIG_SPARC64" = "y" ]; then
> +if [ "$CONFIG_SND" != "n" -a "$CONFIG_SPARC32" = "y" ] || [ "$CONFIG_SND" != "n" -a "$CONFIG_SPARC64" = "y" ] ; then
> source sound/sparc/Config.in
> fi

That's not right. You can't use '||' in config language. (Try it
with xconfig some time.) You have to either nest if statements or
make use of precedence. Documentation/kbuild/config-language.txt
doesn't actually specify the precedence, but in the Unix test(1) from
which it is derived, AND binds tighter than OR. Thus:

if [ "$CONFIG_SND" != "n" ]; then
if [ "$CONFIG_SPARC32" = "y" -o "$CONFIG_SPARC64" = "y" ]; then
source sound/sparc/Config.in
fi
fi

-- or --

if [ "$CONFIG_SND" != "n" -a "$CONFIG_SPARC32" = "y" -o \
"$CONFIG_SND" != "n" -a "$CONFIG_SPARC64" = "y" ]; then
source sound/sparc/Config.in
fi

This 'if' statement syntax has *got* to go. I posted an incomplete
replacement syntax some time ago, but abandoned it because it appeared
Roman was almost ready to merge his new config stuff....

Peter

2002-10-02 19:30:38

by Sam Ravnborg

[permalink] [raw]
Subject: Re: a problem report

On Wed, Oct 02, 2002 at 01:49:11PM -0500, Peter Samuelson wrote:
> [Sam Ravnborg]
[Snip faulty Config.in patch]

> That's not right. You can't use '||' in config language. (Try it
> with xconfig some time.)

I was aware of this, the only reason why I posted this was to let people
proceed with menuconfig, and to give them a response.

Roman Zippel already posted a correct path, that he also has sent to Linus.
Although I do not understand where the ALSA people are in this game.

> This 'if' statement syntax has *got* to go. I posted an incomplete
> replacement syntax some time ago, but abandoned it because it appeared
> Roman was almost ready to merge his new config stuff....
The 2.5.39 version of Roman's lkc looked really good IMHO, but I think
more people has to get into the discussion before it will be accepted.
I for one is not the right person to comment on the config
language, old or new.

Sam

2002-10-02 20:08:03

by Roman Zippel

[permalink] [raw]
Subject: Re: a problem report

Hi,

On Wed, 2 Oct 2002, Sam Ravnborg wrote:

> The 2.5.39 version of Roman's lkc looked really good IMHO, but I think
> more people has to get into the discussion before it will be accepted.

I'd be happy about a discussion, but if it's not happening soon, it's up
to Linus to accept it.

bye, Roman