Patch to not fork/exec xkbcomp on X Server initialization

[Daniel Stone]

On Wed, Jul 02, 2008 at 01:31:48PM +0930, Peter Hutterer wrote:
> [Haven't looked at the actual patch yet]
> 
> On Wed, Jul 02, 2008 at 02:40:54AM +0300, Daniel Stone wrote:
> > Yes, I was expecting around 50% or so.  If this is tested and working
> > fine for you, I'm perfectly happy for you to recommend this to people
> > who need it, but I'm definitely not merging it.  In theory, xkbcomp
> > should just be xkbcomp(), which hands us back the XkbDescRec it
> > generates while parsing anyway.
> 
> A rewrite may be a better option, but is there any reason we can't put
> paulo's patch into master for the time being? Worse comes to worst it's
> reverted (or removed) when the rewrite happens. Trying to debug xkb startup is
> a pain right now, and removing the fork may just convince my gdb not to die
> the death of a thousand sins.

The thing is that this still forks, so you're still screwed on startup,
and you still have the XKM mess.  All it does is cache the output
somewhere.

> and the thing with rewrites is that they tend to always take longer than
> expected, so if there's an interim solution I'm not the one to complain.

Yeah, I'm still happy to try the smashing-xkbcomp-in-its-current-form-in
thing again.

Cheers,
Daniel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg/attachments/20080702/3e982dc8/attachment.pgp>