Re: make menuconfig - cosmetic question

2001-05-17 Thread Martin.Knoblauch

"Martin.Knoblauch" wrote:
> 
> 
>  Not sure whether this is worth to put into the next release - maybe
> someone can spend two minutes to crosscheck.
> 

 This looks more complete. 

lx/linux > diff -u scripts/Menuconfig.orig scripts/Menuconfig
--- scripts/Menuconfig.orig Thu May 17 17:19:21 2001
+++ scripts/Menuconfig  Thu May 17 18:10:22 2001
@@ -74,7 +74,10 @@
 # - Implemented new functions: define_tristate(), define_int(),
define_hex(),
 #   define_string(), dep_bool().
 #
-
+# 17 May 2001, Martin Knoblauch, <[EMAIL PROTECTED]>
+# - Fix output of comment strings to .config/autoconfig.h so that they
equal
+#   the comments from "make [old]config"
+#

 #
 # Change this to TRUE if you prefer all kernel options listed
@@ -1240,7 +1243,7 @@
}
 
function mainmenu_option () {
-   comment_is_option=TRUE
+   :
}
 
function endmenu () {
@@ -1248,9 +1251,7 @@
}
 
function comment () {
-   if [ "$comment_is_option" ]
-   then
-   comment_is_option=
+
echo>>$CONFIG
echo "#">>$CONFIG
echo "# $1" >>$CONFIG
@@ -1260,7 +1261,6 @@
echo "/*">>$CONFIG_H
echo " * $1" >>$CONFIG_H
echo " */"   >>$CONFIG_H
-   fi
}
 
echo -n "."

Martin
PS: This time without vcard ...
-- 
--
Martin Knoblauch |email:  [EMAIL PROTECTED]
TeraPort GmbH|Phone:  +49-89-510857-309
IT Services  |Fax:+49-89-510857-111
http://www.teraport.de   |Mobile: +49-170-4904759
-
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/



Re: make menuconfig - cosmetic question

2001-05-17 Thread Martin.Knoblauch

Bjorn Wesen wrote:
> 
> While we're on cosmetics... how about imprisonment for the person who
> chose yellow on light grey for the first letters in each option...
> 
> /Bjorn
> 
> On Thu, 17 May 2001, Martin.Knoblauch wrote:
> >  this is most likely just a small issue. If I knew where to look, I
> > would try to fix it and submit a patch :-)
> >
> >  When I diff config files pocessed by "make [old]config" and "make
> > menueconfig", it seems that menuconfig is not writing out some of the
> > "comments" that the other versions do write. This is of course nothing
> > serious, but it ticks me off. Any idea where to look for this glitch?

 Now, changing that color is simple. Just change the value of TAG_KEY_FG
in scripts/lxdialog/colors.h. Unfortunatelly, this is not a question of
cosmetics, but taste. And I do not want to go into that :-)

 After some browsing around the Menuconfig script and not really
understanding how it works :-), I found the line to change. At least now
the config files look identical when processed with menuconfig (except
the fisrt comment, of course):

--- linux/Scripts/Menuconfig.orig Thu May 17 17:19:21 2001
+++ linux/scripts/Menuconfig  Thu May 17 17:17:25 2001
@@ -1250,7 +1250,7 @@
function comment () {
if [ "$comment_is_option" ]
then
-   comment_is_option=
+   comment_is_option=TRUE
echo>>$CONFIG
echo "#">>$CONFIG
echo "# $1" >>$CONFIG

 Not sure whether this is worth to put into the next release - maybe
someone can spend two minutes to crosscheck.

Martin
PS: Sorry for the MIME. Should not have happend :-( Hope it is better
now.
-- 
--
Martin Knoblauch |email:  [EMAIL PROTECTED]
TeraPort GmbH|Phone:  +49-89-510857-309
IT Services  |Fax:+49-89-510857-111
http://www.teraport.de   |Mobile: +49-170-4904759

begin:vcard 
n:Knoblauch;Martin
tel;cell:+49-170-4904759
tel;fax:+49-89-510857-111
tel;work:+49-89-510857-309
x-mozilla-html:FALSE
url:http://www.teraport.de
org:TeraPort GmbH;IT-Services
adr:;;Garmischer Straße 4;München;Bayern;D-80339;Germany
version:2.1
email;internet:[EMAIL PROTECTED]
title:Senior System Engineer
x-mozilla-cpt:;32160
fn:Martin Knoblauch
end:vcard



Re: make menuconfig - cosmetic question

2001-05-17 Thread Keith Owens

On Thu, 17 May 2001 16:07:40 +0200, 
"Martin.Knoblauch" <[EMAIL PROTECTED]> wrote:
>This is a multi-part message in MIME format.

Please turn off MIME when sending to linux-kernel, especially those
useless vcards at the end of your mail.

> When I diff config files pocessed by "make [old]config" and "make
>menueconfig", it seems that menuconfig is not writing out some of the
>"comments" that the other versions do write.

Congratulations, you have found reason #93 for replacing the current
configuration system (CML1) with something better - CML1 has multiple
parsers with subtly different rules and output.  CML2 will fix this, in
kernel 2.5.

-
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/



Re: make menuconfig - cosmetic question

2001-05-17 Thread Bjorn Wesen

While we're on cosmetics... how about imprisonment for the person who
chose yellow on light grey for the first letters in each option... 

/Bjorn

On Thu, 17 May 2001, Martin.Knoblauch wrote:
>  this is most likely just a small issue. If I knew where to look, I
> would try to fix it and submit a patch :-)
> 
>  When I diff config files pocessed by "make [old]config" and "make
> menueconfig", it seems that menuconfig is not writing out some of the
> "comments" that the other versions do write. This is of course nothing
> serious, but it ticks me off. Any idea where to look for this glitch?

-
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/



Re: make menuconfig - cosmetic question

2001-05-17 Thread Bjorn Wesen

While we're on cosmetics... how about imprisonment for the person who
chose yellow on light grey for the first letters in each option... 

/Bjorn

On Thu, 17 May 2001, Martin.Knoblauch wrote:
  this is most likely just a small issue. If I knew where to look, I
 would try to fix it and submit a patch :-)
 
  When I diff config files pocessed by make [old]config and make
 menueconfig, it seems that menuconfig is not writing out some of the
 comments that the other versions do write. This is of course nothing
 serious, but it ticks me off. Any idea where to look for this glitch?

-
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/



Re: make menuconfig - cosmetic question

2001-05-17 Thread Keith Owens

On Thu, 17 May 2001 16:07:40 +0200, 
Martin.Knoblauch [EMAIL PROTECTED] wrote:
This is a multi-part message in MIME format.

Please turn off MIME when sending to linux-kernel, especially those
useless vcards at the end of your mail.

 When I diff config files pocessed by make [old]config and make
menueconfig, it seems that menuconfig is not writing out some of the
comments that the other versions do write.

Congratulations, you have found reason #93 for replacing the current
configuration system (CML1) with something better - CML1 has multiple
parsers with subtly different rules and output.  CML2 will fix this, in
kernel 2.5.

-
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/



Re: make menuconfig - cosmetic question

2001-05-17 Thread Martin.Knoblauch

Bjorn Wesen wrote:
 
 While we're on cosmetics... how about imprisonment for the person who
 chose yellow on light grey for the first letters in each option...
 
 /Bjorn
 
 On Thu, 17 May 2001, Martin.Knoblauch wrote:
   this is most likely just a small issue. If I knew where to look, I
  would try to fix it and submit a patch :-)
 
   When I diff config files pocessed by make [old]config and make
  menueconfig, it seems that menuconfig is not writing out some of the
  comments that the other versions do write. This is of course nothing
  serious, but it ticks me off. Any idea where to look for this glitch?

 Now, changing that color is simple. Just change the value of TAG_KEY_FG
in scripts/lxdialog/colors.h. Unfortunatelly, this is not a question of
cosmetics, but taste. And I do not want to go into that :-)

 After some browsing around the Menuconfig script and not really
understanding how it works :-), I found the line to change. At least now
the config files look identical when processed with menuconfig (except
the fisrt comment, of course):

--- linux/Scripts/Menuconfig.orig Thu May 17 17:19:21 2001
+++ linux/scripts/Menuconfig  Thu May 17 17:17:25 2001
@@ -1250,7 +1250,7 @@
function comment () {
if [ $comment_is_option ]
then
-   comment_is_option=
+   comment_is_option=TRUE
echo$CONFIG
echo #$CONFIG
echo # $1 $CONFIG

 Not sure whether this is worth to put into the next release - maybe
someone can spend two minutes to crosscheck.

Martin
PS: Sorry for the MIME. Should not have happend :-( Hope it is better
now.
-- 
--
Martin Knoblauch |email:  [EMAIL PROTECTED]
TeraPort GmbH|Phone:  +49-89-510857-309
IT Services  |Fax:+49-89-510857-111
http://www.teraport.de   |Mobile: +49-170-4904759

begin:vcard 
n:Knoblauch;Martin
tel;cell:+49-170-4904759
tel;fax:+49-89-510857-111
tel;work:+49-89-510857-309
x-mozilla-html:FALSE
url:http://www.teraport.de
org:TeraPort GmbH;IT-Services
adr:;;Garmischer Straße 4;München;Bayern;D-80339;Germany
version:2.1
email;internet:[EMAIL PROTECTED]
title:Senior System Engineer
x-mozilla-cpt:;32160
fn:Martin Knoblauch
end:vcard



Re: make menuconfig - cosmetic question

2001-05-17 Thread Martin.Knoblauch

Martin.Knoblauch wrote:
 
 
  Not sure whether this is worth to put into the next release - maybe
 someone can spend two minutes to crosscheck.
 

 This looks more complete. 

lx/linux  diff -u scripts/Menuconfig.orig scripts/Menuconfig
--- scripts/Menuconfig.orig Thu May 17 17:19:21 2001
+++ scripts/Menuconfig  Thu May 17 18:10:22 2001
@@ -74,7 +74,10 @@
 # - Implemented new functions: define_tristate(), define_int(),
define_hex(),
 #   define_string(), dep_bool().
 #
-
+# 17 May 2001, Martin Knoblauch, [EMAIL PROTECTED]
+# - Fix output of comment strings to .config/autoconfig.h so that they
equal
+#   the comments from make [old]config
+#

 #
 # Change this to TRUE if you prefer all kernel options listed
@@ -1240,7 +1243,7 @@
}
 
function mainmenu_option () {
-   comment_is_option=TRUE
+   :
}
 
function endmenu () {
@@ -1248,9 +1251,7 @@
}
 
function comment () {
-   if [ $comment_is_option ]
-   then
-   comment_is_option=
+
echo$CONFIG
echo #$CONFIG
echo # $1 $CONFIG
@@ -1260,7 +1261,6 @@
echo /*$CONFIG_H
echo  * $1 $CONFIG_H
echo  */   $CONFIG_H
-   fi
}
 
echo -n .

Martin
PS: This time without vcard ...
-- 
--
Martin Knoblauch |email:  [EMAIL PROTECTED]
TeraPort GmbH|Phone:  +49-89-510857-309
IT Services  |Fax:+49-89-510857-111
http://www.teraport.de   |Mobile: +49-170-4904759
-
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/