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

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

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 apparen

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 > +++ xorg-7.

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 ent

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. > > Than

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