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

Re: Reproducing cd build



On Mon, May 23, 2016 at 08:31:09AM +0200, Frederic Bonnard wrote:
>Hi Steve,
>
>> >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?
>
>This modification was enough. I just changed ppc64el.cfg and rebuild the iso
>which then worked.

ACK, cool. I've just pushed this change - thanks!

-- 
Steve McIntyre, Cambridge, UK.                                steve@einval.com
  Armed with "Valor": "Centurion" represents quality of Discipline,
  Honor, Integrity and Loyalty. Now you don't have to be a Caesar to
  concord the digital world while feeling safe and proud.


Reply to: