From owner-freebsd-small Thu Jun 21 12: 2:41 2001 Delivered-To: freebsd-small@freebsd.org Received: from subtopia.informationwave.net (subtopia.informationwave.net [199.74.235.3]) by hub.freebsd.org (Postfix) with ESMTP id C1C6137B407; Thu, 21 Jun 2001 12:02:31 -0700 (PDT) (envelope-from missnglnk@informationwave.net) Received: by subtopia.informationwave.net (Postfix, from userid 100) id 919991C9A5; Thu, 21 Jun 2001 14:55:02 -0400 (EDT) Date: Thu, 21 Jun 2001 14:55:02 -0400 From: Omachonu Ogali To: Luigi Rizzo Cc: freebsd-small@freebsd.org, freebsd-current@freebsd.org Subject: Re: picobsd and mdconfig Message-ID: <20010621145502.A18389@subtopia.informationwave.net> References: <20010621105232.B15323@subtopia.informationwave.net> <200106211618.SAA50566@info.iet.unipi.it> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <200106211618.SAA50566@info.iet.unipi.it>; from luigi@info.iet.unipi.it on Thu, Jun 21, 2001 at 06:18:47PM +0200 Sender: owner-freebsd-small@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Thu, Jun 21, 2001 at 06:18:47PM +0200, Luigi Rizzo wrote: > would you mind resubmitting a patch for the version of > "picobsd" i committed this morning, also keeping in mind the > following: That patch was based off of your 8:49AM (revision 1.7) commit this morning. > * it would be better to define a variable, say MD, which holds > the string "vn" or "md", and is used in accessing the memory > device, so we have mostly the same script working on both current > and stable with just a one-line change. The method of allocating and releasing a md and vn are different, would it be too much to maintain two different scripts (-stable and -current) until mdconfig replaces its counterpart? > * you are recursively gzipping files in /etc now, so you should > also modify the mfs_tree/etc/rc script which decompresses things; > This might be annoying as gzip is unable to recurse in a > directory tree. Good idea, I missed that. > * re. the CONFIG variable... it does not seem to give problem with > the version i committed this morning. I think there was a bug > in the previous commit, but now it should be ok. On line 336 of the script, you export dead air, resulting in no program being passed to Makefile.conf, my snippet of code at the bottom should be removed and a more appropriate line inserted after line 62 along the lines of: CONFIG=${CONFIG:-config} I missed that portion of that script when I was looking at it this morning. > * i am not sure the way you handle arguments is correct, from your > patch it appears that you ignore THETYPE (first argument) if there > are not two arguments. This is wrong, the second argument is optional. Yeah, that should be $1 not $2. > thanks > luigi > -----------------------------------+------------------------------------- > Luigi RIZZO, luigi@iet.unipi.it . Dip. di Ing. dell'Informazione > http://www.iet.unipi.it/~luigi/ . Universita` di Pisa > TEL/FAX: +39-050-568.533/522 . via Diotisalvi 2, 56126 PISA (Italy) > Mobile +39-347-0373137 > -----------------------------------+------------------------------------- > > To Unsubscribe: send mail to majordomo@FreeBSD.org > with "unsubscribe freebsd-current" in the body of the message -- Omachonu Ogali missnglnk@informationwave.net http://www.informationwave.net To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-small" in the body of the message