On 2015-03-24, Wolfgang Schweer wrote: > On Mon, Mar 23, 2015 at 12:02:05PM -0700, Vagrant Cascadian wrote: >> >> On Tue, Mar 17, 2015 at 10:00:08PM +0100, Wolfgang Schweer wrote: > Confirmed after having started an USB stick installation on real (and > very old) hardware; something like /dev/sdXY is mounted on /cdrom inside > d-i environment. > > Inspired by this observation I tried this patch in the virtualbox > environment (booting from 'cd') and as well on bare metal w/ USB stick: > > --- a/ltsp-client-builder.postinst 2015-02-07 18:30:46.000000000 +0100 > +++ b/ltsp-client-builder.postinst 2015-03-24 16:04:09.510087326 +0100 > @@ -64,7 +64,7 @@ > db_progress STEP 1 > > if [ "$USE_CDROM" != "false" ] && [ ! -f /target/media/cdrom/.disk/info ]; > then > - chroot /target mount /media/cdrom > + mount $(mount | grep cdrom | cut -d' ' -f1 | head -1) /target/media/cdrom > log "mounting /media/cdrom" > fi > ... > Installation succeded in both cases. 'head -1' is most probably > unneeded, added just in case the device is mounted twice. It won't do > any harm though, I guess.
I'll try reworking this patch a bit, as it can stand to be more robust in general, but "fixing" this without understanding what exactly is wrong, and without being able to reproduce the bug on a standard debian-installer install worries me... I speculated that debian-edu installs already had ltsp-server-standalone installed and would result in a different state when ltsp-client-builder invokes apt-install ltsp-server-standalone... but that didn't result in any change with debian-installer rc4 (20150324). Earlier in ltsp-client-builder.postinst it checks for /cdrom/.disk/info, and that's clearly working for fine with debian-edu installs. One obvious difference I've seen so far is debian-edu runs ltsp-client-builder after grub-installer. Do debian-edu installs modify /etc/fstab in a way that would result in /media/cdrom not being mountable? What other differences are there in a debian-edu install? live well, vagrant
signature.asc
Description: PGP signature
