On Mon, 15 Oct 2007 23:55:33 +0200, Hisham Muhammad <[EMAIL PROTECTED]> wrote:
> 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. > Yes, you're right. Commited a fix right now. -- /Jonas Using Opera's revolutionary e-mail client: http://www.opera.com/mail/ _______________________________________________ gobolinux-devel mailing list gobolinux-devel@lists.gobolinux.org http://lists.gobolinux.org/mailman/listinfo/gobolinux-devel
