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

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 scripts

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. Any

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 current

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-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 devd,

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 about

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 was

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

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 Document the