Subject: Re: CVS commit: src/etc
To: Julio M. Merino Vidal <jmmv84@gmail.com>
From: Mike M. Volokhov <mishka@netbsd.org>
List: source-changes
Date: 06/05/2007 13:46:48
Hello,

first of all, I'm sorry you has confronted with those diffilcuties.

:-(

"Julio M. Merino Vidal" <jmmv84@gmail.com> wrote:
> On 09/04/2007, at 20:03, matthew green wrote:
> 
> >
> >   On Mon, Apr 02, 2007 at 12:42:42PM +0000, Mike M. Volokhov wrote:
> >>
> >>Module Name:	src
> >>Committed By:	mishka
> >>Date:		Mon Apr  2 12:42:42 UTC 2007
> >>
> >>Modified Files:
> >>	src/etc: wscons.conf
> >>	src/etc/rc.d: wscons
> >>
> >>Log Message:
> >>Provide yet more functionality to wscons.conf's "setvar" (see
> >>examples in the wscons.conf). Now, you have to specify wscons
> >>control device and put variable name and parameter on separate
> >>fields.
> >
> >   Can you please update wscons.conf(5) accordingly?
> >
> >   I had a "hard" time figuring out why setvar didn't work as  
> >before and
> >   why what the manual page mentioned did not work.
> >
> >
> >is there no way to support the old style usage as well?  it seems
> >poor choice to break a configuration file format...
> 
> It looks like it'd be easy to check if there is just one parameter to  
> setvar and assume the old behavior in that case.  Maybe issue a  
> warning in that case to mention that such syntax is obsolete and will  
> be removed in the future?

Well, since more than two month were already passed and no more
comments were arised I think it would be reasonable to update
wscons.conf man page and reflect new behaviour in UPDATING file.

And for correct upgrading of system with user modified wscons.conf
file the etcupdate(8) should be used.

--
Mishka.


P.S. Yes, yes, I'm already learned the rule - do not commit in
haste or while AFK.  Excuse me again for inconvenience, please.