Skip site navigation (1)Skip section navigation (2)
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>