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

Bug#724931: Please include the patch in git



Andreas Cadhalpun <andreas.cadhalpun@googlemail.com> (2014-03-08):
> I looked more closely and found that commenting out the following in
> 40cdrom fixes the problem:
> 	# Allow apt-cdrom to manage mounting/unmounting CDs in /target
> 	if [ "$cd_mountable" ]; then
> 		rm -f $ROOT/etc/apt/apt.conf.d/00NoMountCDROM
> 
> 		$logoutput umount /target/media/cdrom* || true
> 		$logoutput umount /cdrom || true
> 	fi
> 
> At least I managed to install Debian jessie in a Virtual Machine.
> So if there are no compelling reasons to have this (Note: I don't
> know, what this is supposed to do.), I would suggest removing this
> as a fix for Bug #740673.

Err, no.

We had something that worked well enough for a while, it broke due to a
change in apt-cdrom, which was confirmed to be unintentional. So let's
fix that for now, and *maybe* rethink stuff after that.

I'm not goint to drop random lines, especially not based on “I don't
know what this is supposed to do”…

Mraw,
KiBi.

Attachment: signature.asc
Description: Digital signature


Reply to: