AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:onstor-exch02.onstor.net
NSV:
SSH:
R:<larry.scheer@onstor.com>
MAID:1
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#imap/andys@onstor.net@onstor-exch02.onstor.net/INBOX	0	BB375AF679D4A34E9CA8DFA650E2B04E02215970@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Fri, 10 Aug 2007 13:35:22 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Larry Scheer" <larry.scheer@onstor.com>
Subject: Re: Package scripts to review
Message-ID: <20070810133522.36907dde@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E02215970@onstor-exch02.onstor.net>
References: <BB375AF679D4A34E9CA8DFA650E2B04E02215970@onstor-exch02.onstor.net>
Organization: Onstor
X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.8.20; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Thu, 9 Aug 2007 21:11:58 -0700 "Larry Scheer"
<larry.scheer@onstor.com> wrote:

> Andy, 
>    Check the lists in these two scripts and let me know if there is
> anything to add or jettison.
> 
> I am wondering about binutils in the root fs.  It pulls in some dev
> packages that we probably don't want on a QA or eventually production
> system.
> 
> My workspace is /homes/larrys/src/dev
> 
> Change 24816 by larrys@larrys on 2007/08/02 12:00:34 *pending*
> 
>         Updates to script used to fetch and install debian packages.
>         Source packages are fetched by mk-source-repo
>         Root filesystem created by mk-mipsel-deb-rootfs
>         Reviewed by: andys
> 
> Affected files ...
> 
> ... //depot/dev/linux/pkg-tools/mk-mipsel-deb-rootfs#1 edit
> ... //depot/dev/linux/pkg-tools/mk-source-repo#1 edit

linux/pkg-tools/mk-mipsel-deb-rootfs

     line 4, there is no reason to have this variable.  all you do is
     echo it in one place later.  better to make the usage message a
     function and just call it from wherever it's needed, like line 39

     line 72, perhaps we should have certain packages that are put
     into a 'dev' list that gets added to the regular list if we are
     building a dbg VARIANT.  fakeroot would definitely be one of those.

     line 81, tcpd, unless we are running stuff from inetd, and maybe
     even then, we can probably skip this one.  unless we decide to
     integrate tcpd into our UI it won't ever be useful.  this would
     also eliminate libwrap0.

     line 108, no ftp-ssl.  use wget-ssl package instead.  It handles
     both ftp and http and we need it anyway, so we can skip ftp.
     we can use the alternates method for adding an ftp command.

     There are probably some packages that we explicitly have to
     exclude, otherwise they will be pulled onto the distro.  I found
     these to be necessary:

     EXCL_DEBS="--exclude=cyrus-sasl2-doc"
     EXCL_DEBS="${EXCL_DEBS},tasksel-data"
     EXCL_DEBS="${EXCL_DEBS},tasksel"
     EXCL_DEBS="${EXCL_DEBS},laptop-detect"
     EXCL_DEBS="${EXCL_DEBS},libusb-0.1-4"
     EXCL_DEBS="${EXCL_DEBS},aptitude"

     and then say

     $SUDO debootstrap --verbose --arch mipsel --foreign $ADDL_DEBS \
     	$EXCL_DEBS \ etch \ ${ROOTFS} \ $DEB_URL

linux/pkg-tools/mk-source-repo

     line 35, there should be a way to make this variable generated
     rather than static.

     line 137, remove all vestiges of cyrus-sas* from this file

     line 155, tcp wrappers is used to build ??  i don't see how.

     line 183, method should be http, not ftp



