Re: [PATCH 0/2] Trivial patches for d-i/grub-installer

2017-11-06 Thread John Paul Adrian Glaubitz
On 11/06/2017 02:09 PM, Frank Scheiner wrote: And next time I'll try to make the patches "children" of the cover letter, I didn't anticipate this and now they're all on the same level. :-/ No worries. Eventually you should re-send the patches to debian-boot@l.d.o again as this is the official d

Re: [PATCH 0/2] Trivial patches for d-i/grub-installer

2017-11-06 Thread Frank Scheiner
Hi Adrian, On 11/06/2017 10:54 AM, John Paul Adrian Glaubitz wrote: [...] I hope I got this right and my MUA doesn't make the patches unusable. This looks already *much* better. Thanks a lot for your efforts! I will review your patches later today, I'm currently at work. Of course, take yo

[PATCH 5/5] Adapt grub-install parameters and finally perform GRUB installation. Make CHRP script bootable by OpenFirmware. Configure NVRAM defaults but do not boot automatically.

2017-11-06 Thread Frank Scheiner
The NVRAM is (re)configured explicitly after the call to `grub-install` because `grub-install` uses and configures a wrong OF path (thanks to `ofpathname`). As it doesn't work the way `grub-install` does it currently, I tried to support the NVRAM defaults, which are to boot from the first found tbx

[PATCH 4/5] Create and configure the missing CHRP boot script.

2017-11-06 Thread Frank Scheiner
Instead of the `ofpathname` tool (which is used in `grub-install` internally), the `ofpath` tool is used to get the OF path of the used OFFS partition, as this was the only tool that produced a usable/working output on the tested machines (PowerMac11,2 and 7,3). We can omit the creation of the CHRP

[PATCH 3/5] Handle HFS file systems (format, mount, unmount, permanent mount). Override $bootfs, $bootfstype, $bootdev and $state for NewWorld Power Macs.

2017-11-06 Thread Frank Scheiner
--- debian/changelog | 1 + grub-installer | 94 +++- 2 files changed, 94 insertions(+), 1 deletion(-) diff --git a/debian/changelog b/debian/changelog index 752849e..4c056d3 100644 --- a/debian/changelog +++ b/debian/changelog @@ -7,6 +7,7 @

[PATCH 2/5] Detect and select NewWorld bootstrap partitions (adapted from d-i/yaboot-installer).

2017-11-06 Thread Frank Scheiner
--- debian/changelog| 4 +- debian/grub-installer.templates | 19 ++ grub-installer | 126 +++- 3 files changed, 147 insertions(+), 2 deletions(-) diff --git a/debian/changelog b/debian/changelog index 0f1dc54..752849e 10

[PATCH 1/5] Install hfsutils in-target for NewWorld Power Macs (adapted from d-i/yaboot-installer).

2017-11-06 Thread Frank Scheiner
The installation of the hfsutils package is located in debian/postinst similar to d-i/yaboot-installer, from which the code was adapted. But d-i/grub-installer does all in-target installations from the `grub-installer` script itself. If this is important this can still be moved to `grub-installer`.

[PATCH 0/5] Make d-i/grub-installer work for NewWorld Power Macs

2017-11-06 Thread Frank Scheiner
## Make d-i/grub-installer work for NewWorld Power Macs ## When applied, the following five patches should make d-i/grub-installer work on NewWorld Power Macs. This was tested during Debian installations for the ppc64 architecture in "expert" mode on the following machine types: * PowerMac11,2 (

Re: [PATCH 0/2] Trivial patches for d-i/grub-installer

2017-11-06 Thread John Paul Adrian Glaubitz
Hi Frank! On 11/06/2017 10:45 AM, Frank Scheiner wrote: I think I start with the two trivial patches (which will follow this mail). As both are unrelated to each other and the "Make d-i/grub-installer work for NewWorld Power Macs" patches, they're separated, but as both are trivial patches they

[PATCH 1/2] Trivial patch to get the case statement into the coding style of the grub-installer script.

2017-11-06 Thread Frank Scheiner
--- grub-installer | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/grub-installer b/grub-installer index f9f0761..192be53 100755 --- a/grub-installer +++ b/grub-installer @@ -235,13 +235,13 @@ rootfstype="$(findfstype /)" case $ARCH in powerpc/chrp|powerpc/ch

[PATCH 2/2] Execute EFI related code only for installations of grub-efi* packages.

2017-11-06 Thread Frank Scheiner
--- debian/changelog | 7 +++ grub-installer | 30 +- 2 files changed, 24 insertions(+), 13 deletions(-) diff --git a/debian/changelog b/debian/changelog index 1880d5b..0470484 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,10 @@ +grub-installer

[PATCH 0/2] Trivial patches for d-i/grub-installer

2017-11-06 Thread Frank Scheiner
Hi, I think I start with the two trivial patches (which will follow this mail). As both are unrelated to each other and the "Make d-i/grub-installer work for NewWorld Power Macs" patches, they're separated, but as both are trivial patches they go with the same cover letter. I hope I got this righ