Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 26 Jul 2001 02:44:40 +0200
From:      Sheldon Hearn <sheldonh@starjuice.net>
To:        David Hill <david@phobia.ms>
Cc:        audit@freebsd.org
Subject:   Re: STD*_FILENO patches #2 
Message-ID:  <6413.996108280@axl.seasidesoftware.co.za>
In-Reply-To: Your message of "Tue, 24 Jul 2001 09:47:44 %2B0200." <6134.995960864@axl.seasidesoftware.co.za> 

next in thread | previous in thread | raw e-mail | index | archive | help


On Tue, 24 Jul 2001 09:47:44 +0200, Sheldon Hearn wrote:

> > Ok, I fixed those
> > http://www.phobia.ms/patches/diffs.tar.gz
> 
> I'll take a look at these and commit them some time before Friday.

Right, they're running through a buildworld now.

Some comments:

1) Please submit patches in a uniform way.  Some were context diffs,
   some unified diffs.  Some used full pathnames, some relative
   pathnames.  The patchset wasn't as easy to apply as it could have
   been.

2) I've ommitted your rcorder patch, because rcorder is vendor software.
   We put vendor code in places other than contrib/ just to confuse the
   shit out of our contributors.  Please submit that patch to the NetBSD
   maintainer for rcorder, Luke Mewburn <lukem@wasabisystems.com>.

3) Well spotted on usr.sbin/ctm/ctm_rmail/options.h.  This file is
   already missing all sorts of includes, so adding <unistd.h> would
   have introduced inconsistency. :-)

Thanks for the contribution.

Ciao,
Sheldon.

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




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