On 10/09/2017 02:26 AM, John Paul Adrian Glaubitz wrote:
I'm sorry if I am being a bit harsh here now, but I feel like I'm tilting
at windmills here. It's starting to become frustrating.
Don't worry, I think we all have the same goal, but you have a knowledge
lead we have to catch up a little to get to a common understanding, I
think. :D
E.g. for the grub-installer it was never clear - at least to me - what
exactly didn't work on NewWorld Power Macs. As it did work well for you
on POWER7, there was a good chance that it might work to some degree on
the Power Macs, too.
So although everything I wrote seemed to be pretty obvious to you and
possibly a repetition of what you already knew, it was "surprising" to
me and partly also for Rick, I think.
I believe it's good to have this written down though, so we now know
where things need to be fixed or better created from scratch.
If you are willing to help with that, please perform tests and make
suggestions for what to implement.
`partman-newworld` and `partman-prep` packages should also be added to
the installer ISO, as these allow to choose "NewWorld boot partition"
and the respective value for PReP in the "How to use this partition"
screen of the "Partition disks" step. Some of the `partman-newworld`
functionality is also used in `yaboot-installer` (see [1], it allows
`yaboot-installer` to skip the user confirmation for the
selected/detected NewWorld boot partition IIUC), which could be reused
for `grub-installer` to correctly detect the NewWorld boot partition.
[1]:
https://anonscm.debian.org/cgit/d-i/yaboot-installer.git/tree/debian/postinst?id=94cd300c98904d8c308d996239719df4b0c59d6f#n97
They're both on the Debian Jessie 8.9 netinst installer ISO.
If it's only the "ppc64" arch that is missing, the following patches
could make this work.
partman-newworld
```
diff --git a/check.d/newworld b/check.d/newworld
index 4a81745..0afcdf3 100755
--- a/check.d/newworld
+++ b/check.d/newworld
@@ -6,6 +6,8 @@ ARCH="$(archdetect)"
case $ARCH in
powerpc/powermac_newworld)
;;
+ ppc64/powermac_newworld)
+ ;;
*)
exit 0
;;
diff --git a/debian/control b/debian/control
index c20c1c0..9b2108b 100644
--- a/debian/control
+++ b/debian/control
@@ -10,7 +10,7 @@ Vcs-Git:
https://anonscm.debian.org/git/d-i/partman-newworld.git
Package: partman-newworld
Package-Type: udeb
-Architecture: powerpc
+Architecture: powerpc ppc64
XB-Subarchitecture: powermac_newworld
Depends: ${misc:Depends}, partman-base (>= 114), hfs-modules
Description: partman support for new-world PowerMac boot partitions
```
partman-prep
```
diff --git a/debian/control b/debian/control
index a6c89ec..6f740b6 100644
--- a/debian/control
+++ b/debian/control
@@ -9,7 +9,7 @@ Vcs-Git: https://anonscm.debian.org/git/d-i/partman-prep.git
Package: partman-prep
Package-Type: udeb
-Architecture: powerpc ppc64el
+Architecture: powerpc ppc64el ppc64
XB-Subarchitecture: chrp_ibm chrp_rs6k prep
Depends: ${misc:Depends}, partman-base (>= 114)
Description: Add partman support for PPC PReP boot partitions
```
IMHO it would also make sense to make `hfsutils` a dependency of
`partman-newworld`, but as there's no `hfsutils` udeb available, it
won't work that way, right?
Be we could add code to `debian/postinst` for `partman-newworld` to
install `hfsutils` (as DEB package to the target root FS) just like
yaboot does it in [2] - as I found out yesterday evening.
[2]:
https://anonscm.debian.org/cgit/d-i/yaboot-installer.git/tree/debian/postinst?id=94cd300c98904d8c308d996239719df4b0c59d6f#n61
Thoughts?
