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

Bug#724931: Please include the patch in git



Hi,

On 08.03.2014 19:04, Cyril Brulebois wrote:
Andreas Cadhalpun <andreas.cadhalpun@googlemail.com> (2014-03-08):
I generally agree that it would be good to have an alpha 1 installer
to use for testing regressions of this patch, but on the other hand
using loopmount to install Debian *works*.

Sorry, but until it's been pushed to the masses, I can't blindly trust
that. Many moving parts already, not going to add a critical one when
we already have a (known) regression in apt-setup/apt-cdrom. Especially
not a few days before a release.

I didn't really suggest applying the patch without further testing from someone other than me. I just wanted to express my amazement that loopmount works, while the usual way doesn't.

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.

Best regards,
Andreas


Reply to: