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

Bug#390565: Patch for the graphical installer on PPC boxes



About the "sed -i" thing, this is to keep coherency with other parts of the installer, as frans suggested. I guess it's also simpler asking a tester to uncomment a single line instead of adding a line by hand, which could lead to typos and unreliable test results, when debugging the g-i. I really hope this is enough to make g-i work on all those machines where a success was reported after tweaking properly directfbrc.

Attilio

Sven Luther wrote:
On Sun, Oct 01, 2006 at 10:04:35PM +0200, Attilio Fiandrotti wrote:

One question about this patch ? Why not simply >> the lines to the directfbrc
file, instead of adding them and then uncomenting them ?
Apart from that, this patch seems good to me, will now test it and provide
feedback.

Friendly,

Sven Luther

Index: rootskel-gtk/src/etc/directfbrc
===================================================================
--- rootskel-gtk/src/etc/directfbrc     (revisione 41145)
+++ rootskel-gtk/src/etc/directfbrc     (copia locale)
@@ -1,3 +1,4 @@
no-hardware
screenshot-dir=/var/log
-#disable-module=linux_input #needed for touchpads to work
+#disable-module=linux_input
+#disable-module=radeon


Index: rootskel/src/lib/debian-installer-startup.d/S50directfb-linux-powerpc
===================================================================
--- rootskel/src/lib/debian-installer-startup.d/S50directfb-linux-powerpc       (revisione 0)
+++ rootskel/src/lib/debian-installer-startup.d/S50directfb-linux-powerpc       (revisione 0)
@@ -0,0 +1,9 @@
+#disabling radeon DFB module for all radeon-based cards
+if grep "Radeon" /proc/fb; then
+       sed -i "s/^#disable-module=radeon/disable-module=radeon/" /etc/directfbrc
+fi
+
+#disabling linux_input DFB module except for "PowerBook6,7"
+if grep "PowerBook6,7" /proc/cpuinfo ; then
+       sed -i "s/^#disable-module=linux_input/disable-module=linux_input/" /etc/directfbrc
+fi
Index: rootskel/src/lib/debian-installer-startup.d/Makefile
===================================================================
--- rootskel/src/lib/debian-installer-startup.d/Makefile        (revisione 41155)
+++ rootskel/src/lib/debian-installer-startup.d/Makefile        (copia locale)
@@ -32,7 +32,8 @@

  ifeq ($(DEB_HOST_ARCH_CPU),powerpc)
    files += \
-       S45keyboard-linux-powerpc
+       S45keyboard-linux-powerpc \
+       S50directfb-linux-powerpc
  endif

  ifneq (,$(filter mips mipsel,$(DEB_HOST_ARCH_CPU)))







Reply to: