[lxc-devel] [PATCH 1/3] Fix default package selection
Leonid Isaev
lisaev at umail.iu.edu
Tue Apr 1 00:33:51 UTC 2014
On Mon, 31 Mar 2014 18:34:13 -0500
Serge Hallyn <serge.hallyn at ubuntu.com> wrote:
> Quoting Leonid Isaev (lisaev at umail.iu.edu):
> > Do not cherry-pick packages for the default install to avoid dependency
> > issues. Instead, install the base group modulo blacklisted packages.
> >
> > Signed-off-by: Leonid Isaev <lisaev at umail.iu.edu>
>
> Hi - these look good to me (except for adding bashisms which is
> something we generally try to avoid), but I'm not a user/admin/
But... I am confused. The sheband on this template says "bash", not
"sh" (actually, 14 of 17 templates have #!/bin/bash). Isn't this already
enough reason to not consider other shells?
Moreover, archlinux and its install scripts heavily rely on (modern) bash, so
in the current form (even before these patches) lxc-archlinux will most likely
not work properly with other shells.
Thanks,
L.
> developer on arch linux. Alexander or John, do these 3 look
> good to you?
>
> > ---
> > templates/lxc-archlinux.in | 32 ++++++--------------------------
> > 1 file changed, 6 insertions(+), 26 deletions(-)
> >
> > diff --git a/templates/lxc-archlinux.in b/templates/lxc-archlinux.in
> > index 15283df..5aa9e53 100644
> > --- a/templates/lxc-archlinux.in
> > +++ b/templates/lxc-archlinux.in
> > @@ -47,32 +47,12 @@ default_locale="en-US.UTF-8"
> > default_timezone="UTC"
> > pacman_config="/etc/pacman.conf"
> >
> > -# sort of minimal package set
> > -base_packages=(
> > - "systemd"
> > - "systemd-sysvcompat"
> > - "filesystem"
> > - "coreutils"
> > - "kmod"
> > - "procps"
> > - "psmisc"
> > - "pacman"
> > - "bash"
> > - "cronie"
> > - "iproute2"
> > - "iputils"
> > - "inetutils"
> > - "dhcpcd"
> > - "dnsutils"
> > - "nano"
> > - "grep"
> > - "less"
> > - "gawk"
> > - "sed"
> > - "tar"
> > - "gzip"
> > - "which"
> > -)
> > +# by default, install 'base' except the kernel
> > +pkg_blacklist="linux"
> > +base_packages=()
> > +for pkg in $(pacman -Sqg base); do
> > + [[ "${pkg_blacklist//$pkg/}" = "$pkg_blacklist" ]] &&
> > base_packages+=($pkg) +done
> > declare -a additional_packages
> >
> > # split comma-separated string into an array
> > --
> > 1.8.5.3
> >
> > --
> > Leonid Isaev
> > GnuPG key fingerprint: C0DF 20D0 C075 C3F1 E1BE 775A A7AE F6CB 164B 5A6D
>
>
>
> > _______________________________________________
> > lxc-devel mailing list
> > lxc-devel at lists.linuxcontainers.org
> > http://lists.linuxcontainers.org/listinfo/lxc-devel
>
> _______________________________________________
> lxc-devel mailing list
> lxc-devel at lists.linuxcontainers.org
> http://lists.linuxcontainers.org/listinfo/lxc-devel
--
Leonid Isaev
GnuPG key fingerprint: C0DF 20D0 C075 C3F1 E1BE 775A A7AE F6CB 164B 5A6D
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.linuxcontainers.org/pipermail/lxc-devel/attachments/20140331/412c59d8/attachment.pgp>
More information about the lxc-devel
mailing list