On 10/15/07, Jonas Karlsson <[EMAIL PROTECTED]> wrote:
> CVSROOT:        /sources/goboscripts
> Module name:    tools
> Changes by:     Jonas Karlsson <mohjive>        07/10/15 21:15:58
>
> Modified files:
>         Scripts/bin    : VerifyProgram
>
> Log message:
>         Catch if user passes erronous option combinations
>
> CVSWeb URLs:
> http://cvs.savannah.gnu.org/viewcvs/tools/Scripts/bin/VerifyProgram?cvsroot=goboscripts&r1=1.22&r2=1.23
>
> Patches:
> Index: VerifyProgram
> ===================================================================
> RCS file: /sources/goboscripts/tools/Scripts/bin/VerifyProgram,v
> retrieving revision 1.22
> retrieving revision 1.23
> diff -u -b -r1.22 -r1.23
> --- VerifyProgram       11 Oct 2007 09:05:36 -0000      1.22
> +++ VerifyProgram       15 Oct 2007 21:15:57 -0000      1.23
> @@ -33,6 +33,10 @@
>  Add_Option_Boolean "W" "no-web" "Do not check remote keyserver for keys."
>  Parse_Options "$@"
>
> +[ -z "$(Arg 1)" ] && {
> +   ( Show_Help )
> +   Die "No application specified"
> +}

Why is this necessary? The script already has "helpOnNoArguments=yes"
set. It should have been enough for the option parser to catch that.

-- Hisham
_______________________________________________
gobolinux-devel mailing list
gobolinux-devel@lists.gobolinux.org
http://lists.gobolinux.org/mailman/listinfo/gobolinux-devel

Reply via email to