[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [Git][debian-edu/debian-edu-config][master] 032-edu-pkgs: merge those ten apt-get purge calls into one.

On Mon, Jan 28, 2019 at 08:03:43AM +0100, Petter Reinholdtsen wrote:
> Trying to purge non-installed packages is not a bug in the ISO, it is a
> feature, to ensure some packages are removed no matter which profile is
> selected.

While purging non-installed packages doesn't harm if only a single 
package is concerned, it breaks the LTSP chroot installation in case 
several are to be removed at once including non-installed ones (as you 
already pointed out).

When I tested the too big (~21 GiB) BD image some time ago in offline 
mode this code broke the installation:

in_target apt-get -y purge network-manager wpasupplicant \
    network-manager-openvpn network-manager-pptp network-manager-vpnc

Two out of the three packages on the second line were missing on the BD 
image. Instead of using separate apt-get calls to purge them, I only 
used an additional separate call for those on the second line and added 
'|| true' (inside d-i environment). After selecting LTSP chroot 
installation again from the installer menu the default (combined server) 
installation succeeded.

As the time for these separate calls sums up it seems to make sense to 
first check if the related packages are actually installed and then to 
purge the selected ones with one apt-get call.  

The changed script (commit 0439b8) worked like expected but the code 
might be smarter; please check.


Attachment: signature.asc
Description: PGP signature

Reply to: