[PATCH] Cache xkbcomp output

Keith Packard keithp at keithp.com
Thu Jul 12 15:27:11 PDT 2012


Daniel Stone <daniel at fooishbar.org> writes:

> You need to do at least some rudimentary stat() work so that you'll
> rebuild the keymap if the files it uses changes.

My plan is to simply put that work on the package installer loading new
files into the xkb directories. A simple post-install hook that removed
all of the files in the server's cache directory should make that work

I could make the in-server caching optional if you like, either command
line flag or configure option. Attempting to figure out which files are
relevant to stat seems like a huge effort.

> Aside from that, it's our best hope short of forking xkbcommon, adding
> back some of the bits we removed as pointless, and smashing that into
> the server (volunteers?), so, why not.

Even then, this avoids the cost of compilation, which isn't
insignificant. So, I think we want this regardless. Directly compiling
the file in the X server would avoid a whole pile of security drama,
otherwise it doesn't seem like a huge win (at least according to the
profiles I've seen).

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120712/4daa2b16/attachment-0001.pgp>


More information about the xorg-devel mailing list