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

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: