Re: Reproducing cd build

[Steve McIntyre <steve@einval.com>]

On Fri, May 13, 2016 at 05:28:56PM +0200, Frederic Bonnard wrote:
>Hi,
>I managed to run the 2-step process (debian-cd_info.tar.gz creation
>+ debian-cd netinst iso build) and found the issue is in the grub prefix set up
>when grub-mkimage is done.

Aha!

>diff --git a/build/config/ppc64el.cfg b/build/config/ppc64el.cfg
>index af0afb7..48c1bbe 100644
>--- a/build/config/ppc64el.cfg
>+++ b/build/config/ppc64el.cfg
>@@ -46,7 +46,7 @@ arch_cd_info_dir:
>                KERNEL /install/vmlinux \
>                INITRD /install/initrd.gz \
>        < $(GRUB_CFG_CDROM) > $(TEMP_CD_INFO_DIR)/boot/grub/grub.cfg
>-       grub-mkimage -O powerpc-ieee1275 -p '(ieee1275/cdrom)/boot/grub' \
>+       grub-mkimage -O powerpc-ieee1275 -p '()/boot/grub' \
>                -o $(TEMP_CD_INFO_DIR)/boot/grub/powerpc.elf \
>                $(GRUB_MODULES) $(GRUB_MODULES_CDROM)
>        cp -p /usr/lib/grub/powerpc-ieee1275/bootinfo.txt \
>
>On PowerVM if no devalias is defined for cdrom in the OF, the current code fails.
>With the change it will work. I guess that can help also in case the debian installer
>is ran from a USB key (PowerVM and even qemu from SLOF).
>I saw afterwards that this has been fixed a long time ago in Ubuntu :
>https://bugs.launchpad.net/ubuntu/+source/grub2/+bug/1334793
>But it hasn't been pulled in debian :( ; should it be pulled from Ubuntu to keep
>credits or should I send a patch ?

Hmmm. Is there a patch to grub2 as well, or just the change in d-i
build config?

-- 
Steve McIntyre, Cambridge, UK.                                steve@einval.com
"I've only once written 'SQL is my bitch' in a comment. But that code 
 is in use on a military site..." -- Simon Booth