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

Re: New discussion: ppc64 installer -- ext2 /boot partition to keep yaboot happy.



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?


Reply to: