[Bug 136962] Re: Session file left after uninstall breaks X

2007-09-08 Thread Chris Halse Rogers
xserver-xgl (1:1.1.99.1~git20070727-0ubuntu2) gutsy; urgency=low * debian/xserver-xgl-notification.update-notifier + Inform users that we start Xgl automatically, so previously set up Xgl sessions may not work correctly. * Remove old Xsession.d file on install, if it hasn't been modi

[Bug 136962] Re: Session file left after uninstall breaks X

2007-09-08 Thread Chris Halse Rogers
This is a once-off preinst - in this version we change from 00xserver- xgl_start-server to 98xserver-xgl_start-server. Since these are both conf files, dpkg will not automatically remove the old 00... file. If the user has not altered this file we're happy to remove it. If the user has modified

[Bug 136962] Re: Session file left after uninstall breaks X

2007-09-07 Thread Mario Limonciello
I haven't actually applied/built/tested on my machine, but I see one more thing that sticks out to possibly cause trouble. --- xserver-xgl-1.1.99.1~git20070727.orig/debian/xserver-xgl.preinst +++ xserver-xgl-1.1.99.1~git20070727/debian/xserver-xgl.preinst @@ -0,0 +1,17 @@ +#!/bin/sh -e + +OLD_CONF

[Bug 136962] Re: Session file left after uninstall breaks X

2007-09-05 Thread Chris Halse Rogers
Additionally, the debdiff adds -nolisten tcp and +xinerama to the xgl options: The first because Xgl listens on TCP by default, and we don't want that, the second because it should allow people to use the Xinerama hints for multi-head, and it doesn't break anything on single-head configurations.

[Bug 136962] Re: Session file left after uninstall breaks X

2007-09-05 Thread Chris Halse Rogers
Here's an updated debdiff: Review points: 1) postinst now checks for the existence of $unud, and does nothing if it doesn't exist. It didn't seem important enough to add a dependency on update-notifier | adept-notifier 2) $unud is actually defined in postinst (stupid ctrl-k!) 3) It seems the co

[Bug 136962] Re: Session file left after uninstall breaks X

2007-09-04 Thread Mario Limonciello
Okay, in looking this over, I see a few things that you will want to improve upon. 1) If your shipping an update notifier notification that is installed to $unud unconditionally, you need to depend upon update-notifier | adept-notifier. If you don't want to depend on update-notifier, check if t

[Bug 136962] Re: Session file left after uninstall breaks X

2007-09-04 Thread Chris Halse Rogers
Here's a debdiff reworking the Xsession.d configuration stuff. Since it appears that fglrx, at least, wants to put some setup before anything else in Xsession.d, we add Xgl-session to the top of the STARTUP stack. It also adds an upgrade-notifier item, to inform users that we now start xgl automat