Date: Wed, 02 Mar 2005 13:38:35 -0700 From: Scott Long <scottl@samsco.org> To: obrien@FreeBSD.org Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/aac aac.c Message-ID: <4226244B.5070601@samsco.org> In-Reply-To: <20050302201158.GA82598@dragon.nuxi.com> References: <200503020913.j229DBBi054947@repoman.freebsd.org> <4225D96F.9090803@samsco.org> <20050302201158.GA82598@dragon.nuxi.com>
next in thread | previous in thread | raw e-mail | index | archive | help
David O'Brien wrote: > On Wed, Mar 02, 2005 at 08:19:11AM -0700, Scott Long wrote: > >>David E. O'Brien wrote: >> >>>obrien 2005-03-02 09:13:11 UTC >>> >>> FreeBSD src repository >>> >>> Modified files: (Branch: RELENG_5) >>> sys/dev/aac aac.c >>> Log: >>> MFC: rev 1.102: Put some of the probe messages under bootverbose. >>> >>> Revision Changes Path >>> 1.101.2.4 +16 -15 src/sys/dev/aac/aac.c >> >>I don't recall there being an 'MFC after' message on the original >>commit, nor do I recall you contacting the maintainer to see if he >>wanted this merged. In fact, I DO NOT WANT IT MERGED. Thanks. > > > It has been discussed several times that our boot messages have grown too > verbose, and you have taken a position on IRC that they should be > reduced. Why should we have HEAD and RELENG_5 diverged in this way? Why > as a driver developer would you want less information from branch where > the development takes place? > The agreement that was discussed publically and privately was that we'd target this for 6.x, not 5.x. I wanted the aac messages in 5.x bacause, coincidentally, knowing the aac firmware is very important right now. In any case, I think that a lot of what you're doing is very good and I'm glad that you're doing it. But you still need to respect the maintainer relationship that exists on much of the code that you're touching. A simple query to the original others of many of these patches would go a long way to making people more comfortable and avoiding mistakes. Scott
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4226244B.5070601>