Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 6 Feb 2001 11:42:34 +1030
From:      Greg Lehey <grog@lemis.com>
To:        Darren Reed <darrenr@FreeBSD.org>, Darren Reed <darrenr@reed.wattle.id.au>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, asmodai@wxs.nl
Subject:   Re: cvs commit: src/sys/netinet fil.c ip_auth.c ip_compat.h ip_fil.h ip_frag.c ip_frag.h ip_ftp_pxy.c ip_nat.c ip_nat.h ip_proxy.h ip_rcmd_pxy.c ip_state.c ipl.h mlfk_ipl.c
Message-ID:  <20010206114234.C45171@wantadilla.lemis.com>
In-Reply-To: <20010205121107.E95301@wantadilla.lemis.com>; from grog@lemis.com on Mon, Feb 05, 2001 at 12:11:07PM %2B1030
References:  <20010205181907.V462@wantadilla.lemis.com> <200102051642.DAA10687@avalon.reed.wattle.id.au> <200102041424.f14EOWL31289@freefall.freebsd.org> <20010205121107.E95301@wantadilla.lemis.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tuesday,  6 February 2001 at  3:42:32 +1100, Darren Reed wrote:
> In some email I received from Greg Lehey, sie wrote:
>> On Monday,  5 February 2001 at  8:42:23 +0100, Jeroen Ruigrok/Asmodai wrote:
>>> -On [20010205 03:00], Greg Lehey (grog@lemis.com) wrote:
>>>> In this file, none of the last 4 fixes seem to have made it to
>>>> -STABLE, although a number of these conflicts are what I would call
>>>> "bugs".  Do you intend to do something about this?
>>>
>>> Last time I asked Darren wanted to do that himself, otherwise it would
>>> have been MFC'd already, like I did for Sheldon's (g)awk.
>>
>> Darren, comments?  I'd personally prefer you did it, but somebody
>> needs to do so.  One of the bugs unfixed in -STABLE caused me
>> considerable problems a few weeks back.
>
> Sorry, could you resend the original email which talks about the issue?
> I missed it in my daily purge of freebsd cvs email...

Attached.  Recall, though, that committers are expected to read the
commit logs, especially when they relate to their area of activity :-)

> or is there a PR where this information is available?

No, it was just a reply to your message.  Here it is again.

Greg

On Monday,  5 February 2001 at 12:11:07 +1030, Greg Lehey wrote:
> On Sunday,  4 February 2001 at  6:24:32 -0800, Darren Reed wrote:
>> darrenr     2001/02/04 06:24:32 PST
>>
>>   Modified files:
>>     sys/netinet          fil.c ip_auth.c ip_compat.h ip_fil.h
>>                          ip_frag.c ip_frag.h ip_ftp_pxy.c ip_nat.c
>>                          ip_nat.h ip_proxy.h ip_rcmd_pxy.c
>>                          ip_state.c ipl.h mlfk_ipl.c
>>   Log:
>>   fix conflicts
>
> I've had a look at the log for ip_state.c.  Over the last year, you
> have put in 7 updates and changed well over 1000 lines of code.  The
> commit logs all look the same: "fix conflicts".  Can you be a little
> more explicit?
>
> In this file, none of the last 4 fixes seem to have made it to
> -STABLE, although a number of these conflicts are what I would call
> "bugs".  Do you intend to do something about this?
>
> Greg
>
>>   Revision  Changes    Path
>>   1.23      +72 -15    src/sys/netinet/fil.c
>>   1.21      +2 -2      src/sys/netinet/ip_auth.c
>>   1.13      +17 -5     src/sys/netinet/ip_compat.h
>>   1.18      +5 -2      src/sys/netinet/ip_fil.h
>>   1.14      +10 -12    src/sys/netinet/ip_frag.c
>>   1.11      +3 -2      src/sys/netinet/ip_frag.h
>>   1.17      +22 -9     src/sys/netinet/ip_ftp_pxy.c
>>   1.20      +99 -55    src/sys/netinet/ip_nat.c
>>   1.14      +6 -5      src/sys/netinet/ip_nat.h
>>   1.9       +3 -2      src/sys/netinet/ip_proxy.h
>>   1.8       +3 -3      src/sys/netinet/ip_rcmd_pxy.c
>>   1.20      +11 -13    src/sys/netinet/ip_state.c
>>   1.14      +2 -2      src/sys/netinet/ipl.h
>>   1.8       +13 -1     src/sys/netinet/mlfk_ipl.c
--
Finger grog@lemis.com for PGP public key
See complete headers for address and phone numbers


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010206114234.C45171>