Pavlin Radoslavov <[EMAIL PROTECTED]> wrote: > Ben, > > Thank you for the patch. I just committed it to CVS. > Regarding "xorpsh -e" it appears a bug, hence please submit a > Bugzilla entry.
I just submitted Bugzilla entry so the issue is not lost: http://bugzilla.xorp.org/bugzilla/show_bug.cgi?id=763 Pavlin > Ben Greear <[EMAIL PROTECTED]> wrote: > > > Ben Greear wrote: > > > I believe xorpsh -e does not work: > > > > > > [EMAIL PROTECTED] bin]# ./xorpsh -e > > > created new heap in find_heap, ptr: 0x8254210 > > > [ 2008/07/23 13:11:24 WARNING xorpsh LIBXORP ] read error: _fd: 22 > > > offset: 0 total-len: 4 error: Connection refused > > > [ 2008/07/23 13:11:24 WARNING xorpsh LIBXORP ] read error: _fd: 22 > > > offset: 0 total-len: 4 error: Connection refused > > > .... > > > > > > I would expect it to abort immediately. > > > > > > Also, xorpsh -h exits with error code 1 instead of zero. > > > > > > This matters because I'm trying to run an automated test to > > > see if xorp can start (ie, libraries are all configured right, etc). > > > > > > [EMAIL PROTECTED] bin]# ./xorpsh -h > > > Usage: xorpsh [options] > > > Options: > > > -c Specify command(s) to execute > > > -e Exit immediately if cannot connect to the rtrmgr > > > -h Display this information > > > -v Print verbose information > > > -t <dir> Specify templates directory > > > -x <dir> Specify Xrl targets directory > > > Defaults: > > > Templates directory := /usr/local/xorp/etc/templates > > > Xrl targets directory := /usr/local/xorp/xrl/targets > > > Print verbose information := false > > > [EMAIL PROTECTED] bin]# echo $? > > > 1 > > > > Here's the trivial patch for this one: > > > > diff --git a/rtrmgr/xorpsh_main.cc b/rtrmgr/xorpsh_main.cc > > index ce9407c..582e5cf 100644 > > --- a/rtrmgr/xorpsh_main.cc > > +++ b/rtrmgr/xorpsh_main.cc > > @@ -877,7 +877,7 @@ main(int argc, char *argv[]) > > case 'h': > > usage(argv[0]); > > display_defaults(); > > - errcode = 1; > > + errcode = 0; > > goto cleanup; > > } > > } > > > > > > > > > > > > > Thanks, > > > Ben > > > > > > > > > -- > > Ben Greear <[EMAIL PROTECTED]> > > Candela Technologies Inc http://www.candelatech.com > > > > _______________________________________________ > > Xorp-hackers mailing list > > [email protected] > > http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/xorp-hackers > > _______________________________________________ > Xorp-hackers mailing list > [email protected] > http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/xorp-hackers _______________________________________________ Xorp-hackers mailing list [email protected] http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/xorp-hackers
