Re: r36274 - in trunk/packages/debian-installer-utils: . debian
On Tue, Apr 11, 2006 at 12:03:12AM +0200, Frans Pop wrote:
> 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);
That is the one that I prefer
> - only use --no-remove when apt-install is called while installing extra
> packages in base-installer;
> - keep the patch as is.
Me
s/361870/361180/
Reply to: