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

Bug#724931: Please include the patch in git



clone 724931 -1
reassign -1 debian-installer
retitle -1 Please add needed modules for ISO loopmount to work in D-I
thanks

Quoting Andreas Cadhalpun (andreas.cadhalpun@googlemail.com):
> Hi,
> 
> I have researched how to include the needed modules in the initrd.
> 
> If I understand it correctly, it would be possible to add the
> modules as dependencies of cdrom-detect.
> But that would not be feasible, because the kernel modules have the
> version of the kernel in their name and therefore the dependencies
> would have to be updated every time a new kernel is released.
> 
> Instead I think the best way is to change the config files for the
> debian-installer [1] that determine what udebs are included in which
> initrd. They reside in installer/build/pkg-lists.
> 
> For the modules needed for loopmount I attached a patch against the
> debian-installer. In this patch I just added the loop-, ext4-, ntfs-
> and udf-modules to all config files that include cdrom-detect. This
> comes close to making them dependencies of cdrom-detect, but it is
> much easier, since in these config files one can use the variable
> ${kernel:Version} and doesn't have to update that manually.
> 
> Please include that patch in the debian-installer and then release
> the new versions of cdrom-detect, apt-setup, hw-detect and
> mountmedia.

Thanks, Andreas.

I therefore clone this bug report and assign it to debian-installer
itself, so that the needed actions are ataken on time there, too, if
we want this feature.

I was already suspecting that having Cyril Brulebois (CC'ed) look
specifically at all this is really needed as this feature is indeed
quite invasive. So, well, let's get his input.

I have the feeling that, given the time you invested in this, we
should have some cinfidence that your proposal  both makes
sense....and will not break D-I.....but someone less optimistic and
confident than me might want to have a closer look...:-)



Attachment: signature.asc
Description: Digital signature


Reply to: