To PR Senders

Brooks Davis brooks at one-eyed-alien.net
Sat Aug 21 16:49:38 UTC 2004


On Sat, Aug 21, 2004 at 01:23:29AM +0300, Giorgos Keramidas wrote:
> On 2004-08-20 22:58, Marc Fonvieille <blackend at freebsd.org> wrote:
> > On Fri, Aug 20, 2004 at 10:33:06AM -0400, Bill Moran wrote:
> > >
> > > I don't have the knowledge to make exact recommendations, but perhaps
> > > a documented rule of thumb along these lines would help:
> > >
> > > "Patches in excess of 200k, or which contain over 50 lines of completely
> > > new content, should not be attached to the PR, but should be placed on
> > > an ftp/web server and the URL included.  Patches below these limits
> > > should always be attached to the PR."
> >
> > I don't think giving a limit based on lines number is a good idea.
> > 200k is too much, from my point of view, maybe 50k could be the
> > maximum... but some may say gzip exists...
> 
> If you really *do* need a recommendation for a size I think that 50k is
> about the size that things would probably start getting seriously annoying
> for dialup users.  Yeah, if a size must be explicitly specified 50k is ok.

FYI, bde want's patches at least up to 100K, probably somewhat larger to
be inlined.  I'd personally, say that whatever he says is definitive (at
least on the soruce says).

-- Brooks

-- 
Any statement of the form "X is the one, true Y" is FALSE.
PGP fingerprint 655D 519C 26A7 82E7 2529  9BF0 5D8E 8BE9 F238 1AD4
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-doc/attachments/20040821/aa54362c/attachment.sig>


More information about the freebsd-doc mailing list