* [RFC PATCH] kconfig: Add option to get the full help text with listnewconfig
@ 2019-10-29 18:17 Laura Abbott
2019-10-29 22:01 ` Randy Dunlap
2019-11-04 2:20 ` Masahiro Yamada
0 siblings, 2 replies; 4+ messages in thread
From: Laura Abbott @ 2019-10-29 18:17 UTC (permalink / raw)
To: Masahiro Yamada; +Cc: Laura Abbott, linux-kbuild, linux-kernel, jcline, dzickus
make listnewconfig will list the individual options that need to be set.
This is useful but there's no easy way to get the help text associated
with the options at the same time. Introduce a new targe
'make extendedlistnewconfig' which lists the full help text of all the
new options as well. This makes it easier to automatically generate
changes that are easy for humans to review. This command also adds
markers between each option for easier parsing.
Signed-off-by: Laura Abbott <labbott@redhat.com>
---
Red Hat has been relying on some external libraries that have a tendency
to break to get the help text for all new config options. I'd really
like an in tree way to get the help text so we can automatically
generate patches for people to reveiw new config options. I'm open to
other approaches that let us script getting the help text as well.
---
scripts/kconfig/Makefile | 5 ++++-
scripts/kconfig/conf.c | 13 ++++++++++++-
2 files changed, 16 insertions(+), 2 deletions(-)
diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
index ef2f2336c469..aaaf1f62300c 100644
--- a/scripts/kconfig/Makefile
+++ b/scripts/kconfig/Makefile
@@ -66,7 +66,9 @@ localyesconfig localmodconfig: $(obj)/conf
# syncconfig has become an internal implementation detail and is now
# deprecated for external use
simple-targets := oldconfig allnoconfig allyesconfig allmodconfig \
- alldefconfig randconfig listnewconfig olddefconfig syncconfig
+ alldefconfig randconfig listnewconfig olddefconfig syncconfig \
+ extendedlistnewconfig
+
PHONY += $(simple-targets)
$(simple-targets): $(obj)/conf
@@ -134,6 +136,7 @@ help:
@echo ' alldefconfig - New config with all symbols set to default'
@echo ' randconfig - New config with random answer to all options'
@echo ' listnewconfig - List new options'
+ @echo ' extendedlistnewconfig - List new options'
@echo ' olddefconfig - Same as oldconfig but sets new symbols to their'
@echo ' default value without prompting'
@echo ' kvmconfig - Enable additional options for kvm guest kernel support'
diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c
index 40e16e871ae2..7aeb77374d9a 100644
--- a/scripts/kconfig/conf.c
+++ b/scripts/kconfig/conf.c
@@ -32,6 +32,7 @@ enum input_mode {
defconfig,
savedefconfig,
listnewconfig,
+ extendedlistnewconfig,
olddefconfig,
};
static enum input_mode input_mode = oldaskconfig;
@@ -434,6 +435,11 @@ static void check_conf(struct menu *menu)
printf("%s%s=%s\n", CONFIG_, sym->name, str);
}
}
+ } else if (input_mode == extendedlistnewconfig) {
+ printf("-----\n");
+ print_help(menu);
+ printf("-----\n");
+
} else {
if (!conf_cnt++)
printf("*\n* Restart config...\n*\n");
@@ -459,6 +465,7 @@ static struct option long_opts[] = {
{"alldefconfig", no_argument, NULL, alldefconfig},
{"randconfig", no_argument, NULL, randconfig},
{"listnewconfig", no_argument, NULL, listnewconfig},
+ {"extendedlistnewconfig", no_argument, NULL, extendedlistnewconfig},
{"olddefconfig", no_argument, NULL, olddefconfig},
{NULL, 0, NULL, 0}
};
@@ -469,6 +476,7 @@ static void conf_usage(const char *progname)
printf("Usage: %s [-s] [option] <kconfig-file>\n", progname);
printf("[option] is _one_ of the following:\n");
printf(" --listnewconfig List new options\n");
+ printf(" --extendedlistnewconfig List new options and help text\n");
printf(" --oldaskconfig Start a new configuration using a line-oriented program\n");
printf(" --oldconfig Update a configuration using a provided .config as base\n");
printf(" --syncconfig Similar to oldconfig but generates configuration in\n"
@@ -543,6 +551,7 @@ int main(int ac, char **av)
case allmodconfig:
case alldefconfig:
case listnewconfig:
+ case extendedlistnewconfig:
case olddefconfig:
break;
case '?':
@@ -576,6 +585,7 @@ int main(int ac, char **av)
case oldaskconfig:
case oldconfig:
case listnewconfig:
+ case extendedlistnewconfig:
case olddefconfig:
conf_read(NULL);
break;
@@ -657,6 +667,7 @@ int main(int ac, char **av)
/* fall through */
case oldconfig:
case listnewconfig:
+ case extendedlistnewconfig:
case syncconfig:
/* Update until a loop caused no more changes */
do {
@@ -675,7 +686,7 @@ int main(int ac, char **av)
defconfig_file);
return 1;
}
- } else if (input_mode != listnewconfig) {
+ } else if (input_mode != listnewconfig && input_mode != extendedlistnewconfig) {
if (!no_conf_write && conf_write(NULL)) {
fprintf(stderr, "\n*** Error during writing of the configuration.\n\n");
exit(1);
--
2.21.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [RFC PATCH] kconfig: Add option to get the full help text with listnewconfig
2019-10-29 18:17 [RFC PATCH] kconfig: Add option to get the full help text with listnewconfig Laura Abbott
@ 2019-10-29 22:01 ` Randy Dunlap
2019-11-04 2:20 ` Masahiro Yamada
1 sibling, 0 replies; 4+ messages in thread
From: Randy Dunlap @ 2019-10-29 22:01 UTC (permalink / raw)
To: Laura Abbott, Masahiro Yamada; +Cc: linux-kbuild, linux-kernel, jcline, dzickus
Hi Laura,
On 10/29/19 11:17 AM, Laura Abbott wrote:
> diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
> index ef2f2336c469..aaaf1f62300c 100644
> --- a/scripts/kconfig/Makefile
> +++ b/scripts/kconfig/Makefile
> @@ -66,7 +66,9 @@ localyesconfig localmodconfig: $(obj)/conf
> # syncconfig has become an internal implementation detail and is now
> # deprecated for external use
> simple-targets := oldconfig allnoconfig allyesconfig allmodconfig \
> - alldefconfig randconfig listnewconfig olddefconfig syncconfig
> + alldefconfig randconfig listnewconfig olddefconfig syncconfig \
> + extendedlistnewconfig
> +
> PHONY += $(simple-targets)
>
> $(simple-targets): $(obj)/conf
> @@ -134,6 +136,7 @@ help:
> @echo ' alldefconfig - New config with all symbols set to default'
> @echo ' randconfig - New config with random answer to all options'
> @echo ' listnewconfig - List new options'
> + @echo ' extendedlistnewconfig - List new options'
This text should also say " and help text" similar to what conf.c does.
> @echo ' olddefconfig - Same as oldconfig but sets new symbols to their'
> @echo ' default value without prompting'
> @echo ' kvmconfig - Enable additional options for kvm guest kernel support'
I can't say that I like the name, but I won't try to paint it a
different color.
--
~Randy
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RFC PATCH] kconfig: Add option to get the full help text with listnewconfig
2019-10-29 18:17 [RFC PATCH] kconfig: Add option to get the full help text with listnewconfig Laura Abbott
2019-10-29 22:01 ` Randy Dunlap
@ 2019-11-04 2:20 ` Masahiro Yamada
2019-11-04 21:31 ` Laura Abbott
1 sibling, 1 reply; 4+ messages in thread
From: Masahiro Yamada @ 2019-11-04 2:20 UTC (permalink / raw)
To: Laura Abbott
Cc: Linux Kbuild mailing list, Linux Kernel Mailing List,
Jeremy Cline, Don Zickus
On Wed, Oct 30, 2019 at 3:17 AM Laura Abbott <labbott@redhat.com> wrote:
>
> make listnewconfig will list the individual options that need to be set.
> This is useful but there's no easy way to get the help text associated
> with the options at the same time. Introduce a new targe
> 'make extendedlistnewconfig' which lists the full help text of all the
> new options as well. This makes it easier to automatically generate
> changes that are easy for humans to review. This command also adds
> markers between each option for easier parsing.
>
> Signed-off-by: Laura Abbott <labbott@redhat.com>
> ---
> Red Hat has been relying on some external libraries that have a tendency
> to break to get the help text for all new config options. I'd really
> like an in tree way to get the help text so we can automatically
> generate patches for people to reveiw new config options. I'm open to
> other approaches that let us script getting the help text as well.
I am not familiar with the workflow in Red Hat.
I cannot propose another approach.
I am not a big fan of the target name.
'extendedlistnewconfig' is so long that it breaks the
alignment of 'make help'.
Maybe 'helpnewconfig' is understandable
although I am open to discussion for a better name.
BTW, did you check how the newly-added 'choice' was displayed?
Its help message is displayed, but the choice has no name.
So, people might be confused what the help message is talking about.
> ---
> scripts/kconfig/Makefile | 5 ++++-
> scripts/kconfig/conf.c | 13 ++++++++++++-
> 2 files changed, 16 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
> index ef2f2336c469..aaaf1f62300c 100644
> --- a/scripts/kconfig/Makefile
> +++ b/scripts/kconfig/Makefile
> @@ -66,7 +66,9 @@ localyesconfig localmodconfig: $(obj)/conf
> # syncconfig has become an internal implementation detail and is now
> # deprecated for external use
> simple-targets := oldconfig allnoconfig allyesconfig allmodconfig \
> - alldefconfig randconfig listnewconfig olddefconfig syncconfig
> + alldefconfig randconfig listnewconfig olddefconfig syncconfig \
> + extendedlistnewconfig
> +
> PHONY += $(simple-targets)
>
> $(simple-targets): $(obj)/conf
> @@ -134,6 +136,7 @@ help:
> @echo ' alldefconfig - New config with all symbols set to default'
> @echo ' randconfig - New config with random answer to all options'
> @echo ' listnewconfig - List new options'
> + @echo ' extendedlistnewconfig - List new options'
> @echo ' olddefconfig - Same as oldconfig but sets new symbols to their'
> @echo ' default value without prompting'
> @echo ' kvmconfig - Enable additional options for kvm guest kernel support'
> diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c
> index 40e16e871ae2..7aeb77374d9a 100644
> --- a/scripts/kconfig/conf.c
> +++ b/scripts/kconfig/conf.c
> @@ -32,6 +32,7 @@ enum input_mode {
> defconfig,
> savedefconfig,
> listnewconfig,
> + extendedlistnewconfig,
> olddefconfig,
> };
> static enum input_mode input_mode = oldaskconfig;
> @@ -434,6 +435,11 @@ static void check_conf(struct menu *menu)
> printf("%s%s=%s\n", CONFIG_, sym->name, str);
> }
> }
> + } else if (input_mode == extendedlistnewconfig) {
> + printf("-----\n");
> + print_help(menu);
> + printf("-----\n");
> +
> } else {
> if (!conf_cnt++)
> printf("*\n* Restart config...\n*\n");
> @@ -459,6 +465,7 @@ static struct option long_opts[] = {
> {"alldefconfig", no_argument, NULL, alldefconfig},
> {"randconfig", no_argument, NULL, randconfig},
> {"listnewconfig", no_argument, NULL, listnewconfig},
> + {"extendedlistnewconfig", no_argument, NULL, extendedlistnewconfig},
> {"olddefconfig", no_argument, NULL, olddefconfig},
> {NULL, 0, NULL, 0}
> };
> @@ -469,6 +476,7 @@ static void conf_usage(const char *progname)
> printf("Usage: %s [-s] [option] <kconfig-file>\n", progname);
> printf("[option] is _one_ of the following:\n");
> printf(" --listnewconfig List new options\n");
> + printf(" --extendedlistnewconfig List new options and help text\n");
> printf(" --oldaskconfig Start a new configuration using a line-oriented program\n");
> printf(" --oldconfig Update a configuration using a provided .config as base\n");
> printf(" --syncconfig Similar to oldconfig but generates configuration in\n"
> @@ -543,6 +551,7 @@ int main(int ac, char **av)
> case allmodconfig:
> case alldefconfig:
> case listnewconfig:
> + case extendedlistnewconfig:
> case olddefconfig:
> break;
> case '?':
> @@ -576,6 +585,7 @@ int main(int ac, char **av)
> case oldaskconfig:
> case oldconfig:
> case listnewconfig:
> + case extendedlistnewconfig:
> case olddefconfig:
> conf_read(NULL);
> break;
> @@ -657,6 +667,7 @@ int main(int ac, char **av)
> /* fall through */
> case oldconfig:
> case listnewconfig:
> + case extendedlistnewconfig:
> case syncconfig:
> /* Update until a loop caused no more changes */
> do {
> @@ -675,7 +686,7 @@ int main(int ac, char **av)
> defconfig_file);
> return 1;
> }
> - } else if (input_mode != listnewconfig) {
> + } else if (input_mode != listnewconfig && input_mode != extendedlistnewconfig) {
> if (!no_conf_write && conf_write(NULL)) {
> fprintf(stderr, "\n*** Error during writing of the configuration.\n\n");
> exit(1);
> --
> 2.21.0
>
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [RFC PATCH] kconfig: Add option to get the full help text with listnewconfig
2019-11-04 2:20 ` Masahiro Yamada
@ 2019-11-04 21:31 ` Laura Abbott
0 siblings, 0 replies; 4+ messages in thread
From: Laura Abbott @ 2019-11-04 21:31 UTC (permalink / raw)
To: Masahiro Yamada
Cc: Linux Kbuild mailing list, Linux Kernel Mailing List,
Jeremy Cline, Don Zickus
On 11/3/19 9:20 PM, Masahiro Yamada wrote:
> On Wed, Oct 30, 2019 at 3:17 AM Laura Abbott <labbott@redhat.com> wrote:
>>
>> make listnewconfig will list the individual options that need to be set.
>> This is useful but there's no easy way to get the help text associated
>> with the options at the same time. Introduce a new targe
>> 'make extendedlistnewconfig' which lists the full help text of all the
>> new options as well. This makes it easier to automatically generate
>> changes that are easy for humans to review. This command also adds
>> markers between each option for easier parsing.
>>
>> Signed-off-by: Laura Abbott <labbott@redhat.com>
>> ---
>> Red Hat has been relying on some external libraries that have a tendency
>> to break to get the help text for all new config options. I'd really
>> like an in tree way to get the help text so we can automatically
>> generate patches for people to reveiw new config options. I'm open to
>> other approaches that let us script getting the help text as well.
>
> I am not familiar with the workflow in Red Hat.
> I cannot propose another approach.
>
>
> I am not a big fan of the target name.
> 'extendedlistnewconfig' is so long that it breaks the
> alignment of 'make help'.
> Maybe 'helpnewconfig' is understandable
> although I am open to discussion for a better name.
>
>
Yes, I agree the name is long. helpnewconfig is fine by me.
> BTW, did you check how the newly-added 'choice' was displayed?
> Its help message is displayed, but the choice has no name.
> So, people might be confused what the help message is talking about.
>
>
Thanks for pointing this out. For what we're looking to do I
think we can work around it.
Thanks,
Laura
>
>
>> ---
>> scripts/kconfig/Makefile | 5 ++++-
>> scripts/kconfig/conf.c | 13 ++++++++++++-
>> 2 files changed, 16 insertions(+), 2 deletions(-)
>>
>> diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
>> index ef2f2336c469..aaaf1f62300c 100644
>> --- a/scripts/kconfig/Makefile
>> +++ b/scripts/kconfig/Makefile
>> @@ -66,7 +66,9 @@ localyesconfig localmodconfig: $(obj)/conf
>> # syncconfig has become an internal implementation detail and is now
>> # deprecated for external use
>> simple-targets := oldconfig allnoconfig allyesconfig allmodconfig \
>> - alldefconfig randconfig listnewconfig olddefconfig syncconfig
>> + alldefconfig randconfig listnewconfig olddefconfig syncconfig \
>> + extendedlistnewconfig
>> +
>> PHONY += $(simple-targets)
>>
>> $(simple-targets): $(obj)/conf
>> @@ -134,6 +136,7 @@ help:
>> @echo ' alldefconfig - New config with all symbols set to default'
>> @echo ' randconfig - New config with random answer to all options'
>> @echo ' listnewconfig - List new options'
>> + @echo ' extendedlistnewconfig - List new options'
>> @echo ' olddefconfig - Same as oldconfig but sets new symbols to their'
>> @echo ' default value without prompting'
>> @echo ' kvmconfig - Enable additional options for kvm guest kernel support'
>> diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c
>> index 40e16e871ae2..7aeb77374d9a 100644
>> --- a/scripts/kconfig/conf.c
>> +++ b/scripts/kconfig/conf.c
>> @@ -32,6 +32,7 @@ enum input_mode {
>> defconfig,
>> savedefconfig,
>> listnewconfig,
>> + extendedlistnewconfig,
>> olddefconfig,
>> };
>> static enum input_mode input_mode = oldaskconfig;
>> @@ -434,6 +435,11 @@ static void check_conf(struct menu *menu)
>> printf("%s%s=%s\n", CONFIG_, sym->name, str);
>> }
>> }
>> + } else if (input_mode == extendedlistnewconfig) {
>> + printf("-----\n");
>> + print_help(menu);
>> + printf("-----\n");
>> +
>> } else {
>> if (!conf_cnt++)
>> printf("*\n* Restart config...\n*\n");
>> @@ -459,6 +465,7 @@ static struct option long_opts[] = {
>> {"alldefconfig", no_argument, NULL, alldefconfig},
>> {"randconfig", no_argument, NULL, randconfig},
>> {"listnewconfig", no_argument, NULL, listnewconfig},
>> + {"extendedlistnewconfig", no_argument, NULL, extendedlistnewconfig},
>> {"olddefconfig", no_argument, NULL, olddefconfig},
>> {NULL, 0, NULL, 0}
>> };
>> @@ -469,6 +476,7 @@ static void conf_usage(const char *progname)
>> printf("Usage: %s [-s] [option] <kconfig-file>\n", progname);
>> printf("[option] is _one_ of the following:\n");
>> printf(" --listnewconfig List new options\n");
>> + printf(" --extendedlistnewconfig List new options and help text\n");
>> printf(" --oldaskconfig Start a new configuration using a line-oriented program\n");
>> printf(" --oldconfig Update a configuration using a provided .config as base\n");
>> printf(" --syncconfig Similar to oldconfig but generates configuration in\n"
>> @@ -543,6 +551,7 @@ int main(int ac, char **av)
>> case allmodconfig:
>> case alldefconfig:
>> case listnewconfig:
>> + case extendedlistnewconfig:
>> case olddefconfig:
>> break;
>> case '?':
>> @@ -576,6 +585,7 @@ int main(int ac, char **av)
>> case oldaskconfig:
>> case oldconfig:
>> case listnewconfig:
>> + case extendedlistnewconfig:
>> case olddefconfig:
>> conf_read(NULL);
>> break;
>> @@ -657,6 +667,7 @@ int main(int ac, char **av)
>> /* fall through */
>> case oldconfig:
>> case listnewconfig:
>> + case extendedlistnewconfig:
>> case syncconfig:
>> /* Update until a loop caused no more changes */
>> do {
>> @@ -675,7 +686,7 @@ int main(int ac, char **av)
>> defconfig_file);
>> return 1;
>> }
>> - } else if (input_mode != listnewconfig) {
>> + } else if (input_mode != listnewconfig && input_mode != extendedlistnewconfig) {
>> if (!no_conf_write && conf_write(NULL)) {
>> fprintf(stderr, "\n*** Error during writing of the configuration.\n\n");
>> exit(1);
>> --
>> 2.21.0
>>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-11-04 21:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-29 18:17 [RFC PATCH] kconfig: Add option to get the full help text with listnewconfig Laura Abbott
2019-10-29 22:01 ` Randy Dunlap
2019-11-04 2:20 ` Masahiro Yamada
2019-11-04 21:31 ` Laura Abbott
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).