Colin Walters <[email protected]> writes:

> For the GTK+ icon cache, it has a "--force" option to do that, but I
> think it's a fine default to only stat() the toplevel directory and exit
> immediately if the cache timestamp is newer.

For xkb cache files, it is sufficient to simply remove them always
when installing new xkb source files. The server will automatically
regenerate them as needed when run.

> What's relevant particularly here is that the cache file write be
> atomic; I'm not seeing in your patch the code to write it, but hopefully
> it is atomic.

The cache file is generated within the server; right now (as always),
the file is written by xkbcomp and read by the X server. If that
operation fails, then the X server removes the file. If the system
crashes while compiling the keymap, then the (potentially damaged) file
will be detected when the X server tries to load it, and it will be
removed and regenerated.

-- 
[email protected]

Attachment: pgpsbWYbDhbYl.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to