Date: Wed, 21 Dec 2011 22:23:08 GMT From: scheidell@FreeBSD.org To: jeff@bovine.net, scheidell@FreeBSD.org, scheidell@FreeBSD.org, freebsd-ports-bugs@FreeBSD.org Subject: Re: ports/160017: [new port] NeoRouter client & server Message-ID: <201112212223.pBLMN8gG092729@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
Synopsis: [new port] NeoRouter client & server State-Changed-From-To: feedback->open State-Changed-By: scheidell State-Changed-When: Wed Dec 21 22:14:02 UTC 2011 State-Changed-Why: back to the pool. The port doesn't really add anything past what neorouter make install does, and has various problems that need to be solved. #1: won't work if you have ipv6 compiled, unless you use two letter interface names #2: ignores the Feature.ini config file #3: it does not fail on incorrect cli options (nrserver -run -dbdir /var/db/neorouter should fail) #4: errors, when they happen are cryptic at best: nrserver -run --dbroot /var/db/neorouter NeoRouter Free [Version 1.5.1.2820] Copyright (C) 2011 NeoRouter Inc. nrserver failed to start due to error -2138973426. These have to be fixed by neorouter. #5, really needs to be installed in ../sbin not bin. needs a REAL daemon mode, not one that requires daemon (my opinion, not required) As for port ready: the RC file should be reworked. This (default) rc file does not allow for command line options, and, at the very least, should allow you to set --dbroot (FreeBSD conventions call for database to be in /var/db dir, not /usr/local/*) So, if someone else wants this, fine. I suggest that the pkg-plist look for this and not delete it: pwd /usr/local/ZebraNetworkSystems/NeoRouter leroi# ls Feature.ini NeoRouter_0_0_1.db Responsible-Changed-From-To: scheidell->freebsd-ports-bugs Responsible-Changed-By: scheidell Responsible-Changed-When: Wed Dec 21 22:14:02 UTC 2011 Responsible-Changed-Why: http://www.freebsd.org/cgi/query-pr.cgi?pr=160017
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201112212223.pBLMN8gG092729>