From owner-freebsd-current Thu Jun 17 16:24:33 1999 Delivered-To: freebsd-current@freebsd.org Received: from allegro.lemis.com (allegro.lemis.com [192.109.197.134]) by hub.freebsd.org (Postfix) with ESMTP id 4CFBF1572A for ; Thu, 17 Jun 1999 16:24:23 -0700 (PDT) (envelope-from grog@freebie.lemis.com) Received: from freebie.lemis.com (freebie.lemis.com [192.109.197.137]) by allegro.lemis.com (8.9.1/8.9.0) with ESMTP id IAA06999; Fri, 18 Jun 1999 08:54:22 +0930 (CST) Received: (from grog@localhost) by freebie.lemis.com (8.9.3/8.9.0) id IAA54978; Fri, 18 Jun 1999 08:54:30 +0930 (CST) Date: Fri, 18 Jun 1999 08:54:29 +0930 From: Greg Lehey To: John Birrell Cc: jake@checker.org, FreeBSD-current@FreeBSD.ORG Subject: Re: vinum in -current Message-ID: <19990618085428.K9893@freebie.lemis.com> References: <19990618082641.C9893@freebie.lemis.com> <199906172335.JAA26367@cimlogic.com.au> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Mailer: Mutt 0.95.4i In-Reply-To: <199906172335.JAA26367@cimlogic.com.au>; from John Birrell on Fri, Jun 18, 1999 at 09:35:21AM +1000 WWW-Home-Page: http://www.lemis.com/~grog X-PGP-Fingerprint: 6B 7B C3 8C 61 CD 54 AF 13 24 52 F8 6D A4 95 EF Organization: LEMIS, PO Box 460, Echunga SA 5153, Australia Phone: +61-8-8388-8286 Fax: +61-8-8388-8725 Mobile: +61-41-739-7062 Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On Friday, 18 June 1999 at 9:35:21 +1000, John Birrell wrote: > Greg Lehey wrote: >> On Thursday, 17 June 1999 at 9:18:27 -0700, Jake Burkholder wrote: >>> Hi, >>> >>> /sys/modules/Makefile: >>> # XXX vinum removed pending cdevsw changes review by grog. >>> >>> will a module from before the change (May 23rd) work with a new kernel? >>> or can the patches that were sent be made available? >> >> Ugh. Somebody (I know who, but I'm not going to put him to shame in >> public) told me that phk had committed the changes. I didn't know >> that jb then went and removed them. I know we've been flaming phk for >> committing changes without reference, but since I was unreachable for >> a long time, I believe this was the correct thing to do. I'll review >> the changes and reenable the build. Sorry for the delay. > > > > Hey, that's not true! > > phk noted in his commit message for all the _other_ stuff that he had > posted vinum and i4b patches to the respective owners for review. I sent > him mail saying that you were travelling and that he probably should > commit the vinum patches without the review. He said he would prefer to > take vinum out of the build instead. He said he would be offline for a > few hours and if I was online, would I go ahead and make the commit. > So I committed just the makefile change to prevent the build from > failing. Check the cvs history and you will find _no_ vinum commits > from me! > > Well, first, I'm not criticising anybody (much). We've had enough unpleasantness in the past, and I'm sure everybody was doing things with the best of intentions. What I saw was: RCS file: /src/ncvs/src/sys/modules/Makefile,v [snip] ---------------------------- revision 1.63 date: 1999/06/02 07:15:17; author: jb; state: Exp; lines: +3 -2 Remove vinum from the build until Greg reviews phk's cdevsw changes. Preferred by: phk (rather than committing the patch without review). ---------------------------- revision 1.62 date: 1999/05/15 06:13:27; author: grog; state: Exp; lines: +2 -5 Reenable vinum build. ---------------------------- revision 1.61 date: 1999/05/13 09:43:29; author: phk; state: Exp; lines: +5 -2 Vinum doesn't compile right now. Looking at it again, I confused 1.61 and 1.63 (which were, in fact, a couple of weeks apart). OK, I have a machine to rebuild (system disk on my test machine is gradually giving up the ghost), and I hope I'll have a working Vinum again later today (Friday). Greg -- See complete headers for address, home page and phone numbers finger grog@lemis.com for PGP public key To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message