Bug#570447: Acknowledgement (x11-common: Optimize speed of Xsession.d scripts)

2010-04-04 Thread Martin Pitt
Hello Julien, Julien Cristau [2010-04-03 19:16 +0200]: we went from [...] to [...] The changelog says this was to avoid testing for xrdb if ~/.Xresources doesn't exist. But we'll keep testing for xrdb anyway, since we'll pretty much always have /etc/X11/Xresources/. So you'll still run the

Bug#570447: Acknowledgement (x11-common: Optimize speed of Xsession.d scripts)

2010-04-03 Thread Julien Cristau
On Thu, Apr 1, 2010 at 10:27:55 +0200, Martin Pitt wrote: Hello Julien, Julien Cristau [2010-03-14 12:04 +0100]: +- Use shell built in type instead of external which to test for + programs. There's no guarantee that /bin/sh has a 'type' built-in, as far as I can tell

Bug#570447: Acknowledgement (x11-common: Optimize speed of Xsession.d scripts)

2010-04-01 Thread Martin Pitt
Hello Julien, Julien Cristau [2010-03-14 12:04 +0100]: +- Use shell built in type instead of external which to test for + programs. There's no guarantee that /bin/sh has a 'type' built-in, as far as I can tell from SUSv3 and policy). I'd suggest command -v, but apparently posh

Bug#570447: Acknowledgement (x11-common: Optimize speed of Xsession.d scripts)

2010-03-14 Thread Julien Cristau
Hi, I'm probably looking at this a bit late, but... On Fri, Feb 19, 2010 at 06:45:58 +0100, Martin Pitt wrote: diff -Nru xorg-7.5+1ubuntu2/debian/changelog xorg-7.5+1ubuntu3/debian/changelog --- xorg-7.5+1ubuntu2/debian/changelog2010-01-26 00:38:13.0 +0100 +++

Bug#570447: Acknowledgement (x11-common: Optimize speed of Xsession.d scripts)

2010-02-19 Thread Brice Goglin
Martin Pitt wrote: Hello again, argh, I just realized that the previous patch would catch commented options as well, sorry for the blunder! This updated patch defines a new function has_options which encapsulates the option parsing, and thus makes the code easier to read. Thanks,

Bug#570447: Acknowledgement (x11-common: Optimize speed of Xsession.d scripts)

2010-02-19 Thread Martin Pitt
Brice Goglin [2010-02-19 12:37 +0100]: How faster is the boot thanks to this patch ? It's nothing serious, of course, in the 0.1 s range. I was just going through all Xsession.d scripts, and some like 90consolekit were quite heavy. So I just cleaned up all of them while I was at it. Now the

Bug#570447: Acknowledgement (x11-common: Optimize speed of Xsession.d scripts)

2010-02-18 Thread Martin Pitt
Hello again, argh, I just realized that the previous patch would catch commented options as well, sorry for the blunder! This updated patch defines a new function has_options which encapsulates the option parsing, and thus makes the code easier to read. Thanks, Martin -- Martin Pitt