Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-20 Thread Brooks Davis
On Fri, Jan 20, 2012 at 03:06:02PM +0400, Eygene Ryabinkin wrote: > Thu, Jan 19, 2012 at 11:27:59AM -0600, Brooks Davis wrote: > > Arguably it should be moved to /libexec since it's not an rc.d > > script and simply uses the framework because it had similar needs > > Well, I fear that some user sc

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-20 Thread Eygene Ryabinkin
Thu, Jan 19, 2012 at 11:27:59AM -0600, Brooks Davis wrote: > On Fri, Jan 13, 2012 at 11:21:48AM +0400, Eygene Ryabinkin wrote: > > The related topic: in the process of grepping for dhclient within > > /etc, I had found that /etc/netstart still wants to invoke it. But it > > will do a lone '/etc/rc

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-19 Thread Brooks Davis
On Fri, Jan 13, 2012 at 11:21:48AM +0400, Eygene Ryabinkin wrote: > The related topic: in the process of grepping for dhclient within > /etc, I had found that /etc/netstart still wants to invoke it. But it > will do a lone '/etc/rc.d/dhclient quietstart' and this will never > be useful in the curr

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-19 Thread John Baldwin
On Wednesday, January 18, 2012 11:19:53 pm Eygene Ryabinkin wrote: > Wed, Jan 18, 2012 at 10:11:04AM -0500, John Baldwin wrote: > > On Wednesday, January 18, 2012 6:29:10 am Eygene Ryabinkin wrote: > > > The attached patch that just changes 'err' to 'echo ...; exit 1' > > > works fine for me. > > >

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-18 Thread Eygene Ryabinkin
Wed, Jan 18, 2012 at 10:11:04AM -0500, John Baldwin wrote: > On Wednesday, January 18, 2012 6:29:10 am Eygene Ryabinkin wrote: > > The attached patch that just changes 'err' to 'echo ...; exit 1' > > works fine for me. > > > > Any views on it? > > Seems ok to me. > > > Anyone can say anything ab

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-18 Thread John Baldwin
On Wednesday, January 18, 2012 6:29:10 am Eygene Ryabinkin wrote: > Fri, Jan 13, 2012 at 11:21:48AM +0400, Eygene Ryabinkin wrote: > > But seems like the whole problem is that I used the err() function > > instead of doing 'echo $errmsg; exit 1'. Such code shouldn't be > > conditionalized for the

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-18 Thread Eygene Ryabinkin
Fri, Jan 13, 2012 at 11:21:48AM +0400, Eygene Ryabinkin wrote: > But seems like the whole problem is that I used the err() function > instead of doing 'echo $errmsg; exit 1'. Such code shouldn't be > conditionalized for the devd, since it is only syslog message that > worried people; devd doesn't

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-12 Thread Eygene Ryabinkin
Thu, Jan 12, 2012 at 02:38:16PM -0500, John Baldwin wrote: > On Thursday, January 12, 2012 9:35:05 am Eygene Ryabinkin wrote: > > True, and it was suggested by Doug Barton. But why bother when > > the 'quiet' keyword is reserved for such cases? > > Hmm, that's not quite how I see this. Either /e

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-12 Thread John Baldwin
On Thursday, January 12, 2012 9:35:05 am Eygene Ryabinkin wrote: > Thu, Jan 12, 2012 at 07:48:28AM -0500, John Baldwin wrote: > > Alternatively, devd could have invoked a wrapper script that only > > ran /etc/rc.d/dhclient if dhcpif was true for the interface. All > > other uses of /etc/rc.d/dhcli

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-12 Thread Eygene Ryabinkin
Thu, Jan 12, 2012 at 07:48:28AM -0500, John Baldwin wrote: > Alternatively, devd could have invoked a wrapper script that only > ran /etc/rc.d/dhclient if dhcpif was true for the interface. All > other uses of /etc/rc.d/dhclient in our system scripts are > conditional on that check. True, and it

Re: svn commit: r230007 - in head: etc etc/rc.d share/man/man8

2012-01-12 Thread John Baldwin
On Thursday, January 12, 2012 1:48:11 am Eygene Ryabinkin wrote: > Author: rea (ports committer) > Date: Thu Jan 12 06:48:11 2012 > New Revision: 230007 > URL: http://svn.freebsd.org/changeset/base/230007 > > Log: > rc.d: document 'quiet' prefix and fix dhclient/devd interaction > > Documen