* Petter Reinholdtsen <[EMAIL PROTECTED]> [080116 14:11]: > [Micah Anderson] > > hotkey-setup shouldn't assume that an xorg.conf file exists at all, > > in fact the X strike force people are making it easier and easier every > > day to run X without a config file as X can figure most of this out > > without one (see recent blog post on planet by david nusinow for example). > > Good catch. Does this patch solve the issue?
Fast response! Actually I failed to mention this in the bug report, but the error was observed on package installation, so I thought that it was a post-init or something. Now that I see your patch, it was was calling the initscript that brought on the error. I applied the patch and it seems to work. Thanks, micah > diff -Nru /tmp/eGMtg7Df1c/hotkey-setup-0.1/debian/init.d > /tmp/Lgxpw5ijda/hotkey-setup-0.1/debian/init.d > --- /tmp/eGMtg7Df1c/hotkey-setup-0.1/debian/init.d 2008-01-10 > 21:00:34.000000000 +0100 > +++ /tmp/Lgxpw5ijda/hotkey-setup-0.1/debian/init.d 2008-01-16 > 23:08:03.000000000 +0100 > @@ -20,8 +20,14 @@ > THINKPAD_PROC_HOTKEY=/proc/acpi/ibm/hotkey > THINKPAD_KEYS=/usr/sbin/thinkpad-keys > > +xorg_driver() { > + if [ -e /etc/X11/xorg.conf ] ; then > + sed -n -e '/^[ \t]*section[ \\t]*"device"/I,/^[ \t]*endsection/I{/^[ > \t]*driver[ \t]*/I{s/^[ \t]*driver[ \t]*"*//I;s/"*[ \t]*$//;p}}' > /etc/X11/xorg.conf > + fi > +} > + > do_video () { > - VIDEO=`sed -n -e '/^[ \t]*section[ \\t]*"device"/I,/^[ > \t]*endsection/I{/^[ \t]*driver[ \t]*/I{s/^[ \t]*driver[ \t]*"*//I;s/"*[ > \t]*$//;p}}' /etc/X11/xorg.conf` > + VIDEO=`xorg_driver` > case $VIDEO in > intel|ati|radeon) > for x in /proc/acpi/video/*/DOS; do > @@ -39,7 +45,7 @@ > . /usr/share/hotkey-setup/ibm.hk > modprobe thinkpad-acpi > > - VIDEO=`sed -n -e '/^[ \t]*section[ \\t]*"device"/I,/^[ > \t]*endsection/I{/^[ \t]*driver[ \t]*/I{s/^[ \t]*driver[ \t]*"*//I;s/"*[ > \t]*$//;p}}' /etc/X11/xorg.conf` > + VIDEO=`xorg_driver` > case $VIDEO in > intel|ati|radeon) > echo 0xffffff > $THINKPAD_PROC_HOTKEY > > Happy hacking, > -- > Petter Reinholdtsen
signature.asc
Description: Digital signature