ubuntu-x-swat team mailing list archive

[Ryan Hamilton <rthamilt@xxxxxxxx>]

Hi again,

  There's a typo in the latest patch that keeps this from it's intended
result.  In a vanilla 9.04 install, there is an extra space in the added
line that causes the grep statement to fail and still default to vesa
for the driver.  On my system failsafeDexconf read:

VBOX_VIDEO-$(grep -e "^vbox " /proc/modules || true)

where it should read

VBOX_VIDEO-$(grep -e "^vbox" /proc/modules || true)

It'd also be useful to change the modes available as well, using the
same that are already written in for QEMU.  I've changed on my system
the line

if [ -n "$QEMU_KVM" ]; then

to

if [ -n "$QEMU_KVM" ] || [ -n "$VBOX_VIDEO" ]; then

and added an additional "1280x1024" mode so I can use it full screen on
my desktop.

Thanks for putting in the previous changes!  This is my first
contribution to Ubuntu, and it was cool to see it changed so quickly.

-- 
failsafeX has hooks for QEMU, but not VirtualBox
https://bugs.launchpad.net/bugs/319373
You received this bug notification because you are a member of Ubuntu-X,
which is subscribed to xorg in ubuntu.