Kenshi Muto wrote:
> Takeshi Yaegashi, who is my friend, succeeded to install Debian
> on SONY PlayStation 3 (PS3). PS3 is PPC64 architecture and
> some Linux distributions and NetBSD are already ported for PS3.
Very excellent, I've added a link to this to the Debian wiki too.
> SVN diff against current d-i trunk (still under development,
> he is carefully changing d-i to keep small):
> http://www.keshi.org/moin/moin.cgi/PS3/Debian/Installer/Porting
>
> Although I haven't PS3, it's very interesting that d-i merges
> his PS3 PPC specific patch for post-Etch.
I agree, and maybe some more of us will have ps3's by then. :-)
If Takeshi is interested in commit access to the d-i subversion repository,
I feel that this easily justifies it. Then he could keep his patches in
a branch and work on merging things in as appropriate.
I've taken an initial look at the patch, and I feel the following hunks
could be applied as-is without problems to d-i trunk, although we might
prefer to wait until there is a post-etch branch to avoid
destabilisation. Not that I feel there's much risk, most of these changes
are minor.
+++ installer/build/boot/powerpc/kboot.conf
+++ installer/build/config/powerpc/ps3pf/monolithic.cfg
+++ installer/build/config/powerpc.cfg
(would need a minor modification to have ps3pf commented out for
now)
+++ packages/hw-detect/hw-detect.sh
+++ packages/rootskel/src/lib/debian-installer-startup.d/S02ps3pf-linux-powerpc
+++ packages/rootskel/src/lib/debian-installer-startup.d/Makefile
+++ packages/libdebian-installer/src/system/subarch-powerpc-linux.c
+++ packages/arch/powerpc/kboot-installer/*
(can't be uploaded yet, and probably not yet a candidate for
translation, but fine to check it into our svn)
+++ packages/partman/partman-base/definitions.sh
(second hunk only)
+++ packages/base-installer/kernel/powerpc.sh
(it would be good to also add a test case with a copy of the ps3's
cpuinfo file)
I don't understand the following changes:
--- installer/build/pkg-lists/monolithic/powerpc.cfg (revision 43755)
+++ installer/build/pkg-lists/monolithic/powerpc.cfg (working copy)
@@ -8,5 +8,5 @@
dhcp3-client-udeb -
eject-udeb
# Used by yaboot-installer
-parted-udeb
-os-prober
+parted-udeb ?
+os-prober ?
As far as I can see, these should be available for the ps3 image too,
so no need to mark them as optional?
+++ packages/partman/partman-base/definitions.sh (working copy)
@@ -771,6 +771,8 @@
enable_swap () {
local swaps dev num id size type fs path name method
local startdir="$(pwd)"
+ # XXX
+ return 0
# do swapon only when we will be able to swapoff afterwards
[ -f /proc/swaps ] || return 0
swaps=''
If there's no swap on this machine, there are probably better ways to
handle that. :-)
--- packages/debian-installer-utils/apt-install (revision 43755)
+++ packages/debian-installer-utils/apt-install (working copy)
@@ -30,7 +30,7 @@
export DEBIAN_FRONTEND
ERRCODE=0
-log-output -t apt-install chroot /target apt-get -y --no-remove install $@ < /dev/null || ERRCODE=$?
+log-output -t apt-install chroot /target apt-get -y --force-yes --no-remove install $@ < /dev/null || ERRCODE=$?
chroot_cleanup
if [ "$ERRCODE" != 0 ]; then
This change seems odd, especially since the patch does not use apt-install.
--
see shy jo
Attachment:
signature.asc
Description: Digital signature