Date: Sun, 01 Apr 2007 02:59:53 -0700 From: Colin Percival <cperciva@freebsd.org> To: Kostik Belousov <kostikbel@gmail.com> Cc: cvs-src@FreeBSD.org, Kris Kennaway <kris@FreeBSD.org>, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/pci if_vr.c Message-ID: <460F8299.20307@freebsd.org> In-Reply-To: <20070401091501.GK18105@deviant.kiev.zoral.com.ua> References: <20070401070403.16724517E2@repoman.freebsd.org> <20070401091501.GK18105@deviant.kiev.zoral.com.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
Kostik Belousov wrote: > On Sun, Apr 01, 2007 at 07:03:43AM +0000, Kris Kennaway wrote: >> Modified files: >> sys/pci if_vr.c >> Log: >> Fix a search-and-replace bug that slipped through my diff review of the >> Giant pushdown. > > Did you ever test _your_ changes, at least with half of the care you > put into finding bugs in commits of others ? > > You fixed the if_vr.c, but not the flag definition in if_vrreg.h, thus > breaking the build ! I pointed this out to Kris a couple of hours ago, but by that point the CVS meisters were already partway through undoing Kris' commit due to an unrelated issue (accidentally taking files off the vendor branch), so we agreed that it was easiest just to wait for the entire commit to be undone rather than fixing this with another commit. Judging by what I see at http://www.freebsd.org/cgi/cvsweb.cgi/src/sys/, it looks like everything has been reverted now, so hopefully Kris can go ahead and redo his original commit later today without any of the changes which slipped in by mistake. Colin Percival
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?460F8299.20307>