From owner-freebsd-bugs Sun May 24 10:01:06 1998 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id KAA11800 for freebsd-bugs-outgoing; Sun, 24 May 1998 10:01:06 -0700 (PDT) (envelope-from owner-freebsd-bugs@FreeBSD.ORG) Received: from ns1.yes.no (ns1.yes.no [195.119.24.10]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id KAA11795 for ; Sun, 24 May 1998 10:01:03 -0700 (PDT) (envelope-from eivind@bitbox.follo.net) Received: from bitbox.follo.net (bitbox.follo.net [195.204.143.218]) by ns1.yes.no (8.8.7/8.8.7) with ESMTP id RAA09333; Sun, 24 May 1998 17:00:43 GMT Received: (from eivind@localhost) by bitbox.follo.net (8.8.8/8.8.6) id TAA17495; Sun, 24 May 1998 19:00:30 +0200 (MET DST) Message-ID: <19980524190029.29308@follo.net> Date: Sun, 24 May 1998 19:00:29 +0200 From: Eivind Eklund To: Niall Smart Cc: Studded , ac199@hwcn.org, Ruslan Ermilov , nick@foobar.org, freebsd-bugs@FreeBSD.ORG Subject: Re: Problem reports closed by Poul-Henning Kamp [was: Re: misc/6712] References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Mailer: Mutt 0.89.1i In-Reply-To: ; from Niall Smart on Sun, May 24, 1998 at 05:21:38PM +0100 Sender: owner-freebsd-bugs@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org On Sun, May 24, 1998 at 05:21:38PM +0100, Niall Smart wrote: > > This does _not_ involve a re-handling of the PR - it _only_ involve > > handling of the code. A patch is most often not appropriate to commit > > verbatim; style fixes etc is done before it is committed. It would > > not be efficient to handle this from the PR. > > Style fixes are exactly the type of duplication of effort I'm talking > about, This effort is _not_ duplicated if the merge is done from source. Effort mighthave to be duplicated if the change is committed directly to -current and -stable - any subsequent style-fixes (and we're not perfect, so comments on the style of code we commit _do_ come). > if the original PR reviewer can commit a patch to -current and > -stable simultaneously then that is what should happen. Agreed, for really trivial changes where the committer at least have the ability to do a compile-test. > If he doesn't have time to do both, or if the patch requires a > cooling off period in -current then the PR should be shunted into a > new category instead of doing it in -current only and then closing > it. The only effect of this is that we have a new category were some of the changes that should be merged to -stable are, along with some changes that have been merged. This is will not be of much help unless somebody take the time to go through this category and keep it correct (ie, at least periodically go through it and remove changes that already have been committed), and some committers take responsibility for making sure that the changes are merged correctly. Frankly, I'm not certain that a pass through this base will be significantly less work than doing a merge-run - you have to do more-or-less the equivalent of a merge-run for those areas of the source touched, anyway. The advantage is that somebody without commit privileges could keep the log clean. Eivind. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-bugs" in the body of the message