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

Re: r36274 - in trunk/packages/debian-installer-utils: . debian



On Monday 10 April 2006 23:47, Frans Pop wrote:
> --- trunk/packages/debian-installer-utils/apt-install	(original)
> +++ trunk/packages/debian-installer-utils/apt-install
>  export DEBIAN_FRONTEND
>
>  ERRCODE=0
> -log-output -t apt-install chroot /target apt-get -y install
>  $@ < /dev/null || ERRCODE=$?
> +log-output -t apt-install chroot /target apt-get -y --no-remove install
>  $@ < /dev/null || ERRCODE=$? 
> chroot_cleanup
>
>  if [ "$ERRCODE" != 0 ]; then

I'm not completely sure about this patch. In theory it could break 
installs when apt-setup is called after base-installer has run if sources 
added by apt-setup contain packages with e.g. ABI upgrades which makes it 
necessary to remove the old lib while installing the new.

So, what's the best option:
- ignore the issue in #361870 and just let apt-install remove the kernel
  (i.e. revert this change);
- only use --no-remove when apt-install is called while installing extra
  packages in base-installer;
- keep the patch as is.

Attachment: pgpF0rzrcjipS.pgp
Description: PGP signature


Reply to: