| Message ID | 1418331304-4821-2-git-send-email-olof@lixom.net (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi, Olof Not that I care, but what does this patch improve? On Thu, 11 Dec 2014 12:55:04 -0800 Olof Johansson <olof@lixom.net> wrote: > It's usual enough for help text that it makes sense to support it. > > Signed-off-by: Olof Johansson <olof@lixom.net> > --- > scripts/kconfig/merge_config.sh | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/scripts/kconfig/merge_config.sh b/scripts/kconfig/merge_config.sh > index 2ab91b9..0c89537 100755 > --- a/scripts/kconfig/merge_config.sh > +++ b/scripts/kconfig/merge_config.sh > @@ -28,11 +28,11 @@ trap clean_up HUP INT TERM > > usage() { > echo "Usage: $0 [OPTIONS] [CONFIG [...]]" > - echo " -h display this help text" > - echo " -m only merge the fragments, do not execute the make command" > - echo " -n use allnoconfig instead of alldefconfig" > - echo " -r list redundant entries when merging fragments" > - echo " -O dir to put generated output files" > + echo " -h or -? display this help text" > + echo " -m only merge the fragments, do not execute the make command" > + echo " -n use allnoconfig instead of alldefconfig" > + echo " -r list redundant entries when merging fragments" > + echo " -O dir to put generated output files" > } > > MAKE=true > @@ -52,7 +52,7 @@ while true; do > shift > continue > ;; > - "-h") > + "-h"|"-?") > usage > exit > ;; > -- > 1.7.10.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
No objection to the addition of -? if you find it useful.
Acked-by: Darren Hart <dvhart@linux.intel.com>
On Thu, Dec 11, 2014 at 12:55:04PM -0800, Olof Johansson wrote:
> It's usual enough for help text that it makes sense to support it.
It's quite unusual for a *nix utility to recognize '-?'. And -h is
already supported.
Michal
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/scripts/kconfig/merge_config.sh b/scripts/kconfig/merge_config.sh index 2ab91b9..0c89537 100755 --- a/scripts/kconfig/merge_config.sh +++ b/scripts/kconfig/merge_config.sh @@ -28,11 +28,11 @@ trap clean_up HUP INT TERM usage() { echo "Usage: $0 [OPTIONS] [CONFIG [...]]" - echo " -h display this help text" - echo " -m only merge the fragments, do not execute the make command" - echo " -n use allnoconfig instead of alldefconfig" - echo " -r list redundant entries when merging fragments" - echo " -O dir to put generated output files" + echo " -h or -? display this help text" + echo " -m only merge the fragments, do not execute the make command" + echo " -n use allnoconfig instead of alldefconfig" + echo " -r list redundant entries when merging fragments" + echo " -O dir to put generated output files" } MAKE=true @@ -52,7 +52,7 @@ while true; do shift continue ;; - "-h") + "-h"|"-?") usage exit ;;
It's usual enough for help text that it makes sense to support it. Signed-off-by: Olof Johansson <olof@lixom.net> --- scripts/kconfig/merge_config.sh | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)