Re: Adding ppc64el support in debian-cd
On Fri, Oct 03, 2014 at 02:34:42PM +0200, Thomas Schmitt wrote:
> Hi,
>
> Steve McIntyre wrote:
>
> > > I've also just added ppc64el to the daily and weekly build config -
> > > let's see how well it works!
>
> Aurelien Jarno wrote:
>
> > It seems the daily build has worked correctly.
>
> If you mean
> http://cdimage.debian.org/cdimage/daily-builds/daily/arch-latest/ppc64el/iso-cd/debian-testing-ppc64el-netinst.iso
> then it lacks the CHRP (MBR) partition table.
At least I was able to boot it, but maybe SLOF doesn't need it.
> The xorriso command line in /.disk/mkisofs matches this.
> It contains no boot options.
>
> In your patch of Fri, 3 Oct 2014 00:33:01 +0200 i see
>
> +if [ -d CD$N/ppc/bootinfo.txt ] ; then
> + add_mkisofs_opt $CDDIR/../$N.mkisofs_opts "-chrp-boot"
> +fi
>
> which did not come into effect or else would have sabotaged
> the production run with message
> xorriso : FAILURE : -as mkisofs: Unsupported option '-chrp-boot'
-chrp-boot is an option supported by genisoimage, but it looks like it's
not the case of xorriso. Note that this option is also use in the
powerpc build.
> The reason why it did not influence the production run is most
> probably the test option -d for the data file ppc/bootinfo.txt.
Oops.
> How about this:
>
> if [ -f CD$N/ppc/bootinfo.txt ] ; then
> add_mkisofs_opt $CDDIR/../$N.mkisofs_opts "-chrp-boot-part"
> fi
Looks fine. Thanks for the fix.
Aurelien
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
aurelien@aurel32.net http://www.aurel32.net
Reply to: