On Sat, Jan 10, 2009 at 4:43 AM, Daniel Stone <dan...@fooishbar.org> wrote: > On Sat, Jan 10, 2009 at 02:40:04PM +0200, Daniel Stone wrote: >> On Wed, Jan 07, 2009 at 08:27:39AM -0800, Dan Nicholson wrote: >> > On Wed, Jan 7, 2009 at 7:44 AM, Daniel Stone <dan...@fooishbar.org> wrote: >> > > NAK of sorts: I'd like to fix this properly. In particular, hardcoded >> > > paths are not the way. >> > >> > Sure. However, as it stands on both master and 1.6-branch, all the DDX >> > except Xorg start with these defaults. So, they have an invalid >> > default model of "dflt" as far as I can tell. Also, the >> > XKB_BASE_DIRECTORY setting could be dropped from this patch. It's >> > always defined in configure, so the hardcoded /usr/share/X11/xkb >> > should never actually get used. >> > >> > What did you have in mind to fix this properly? >> >> Something like this? > > Or maybe even something like this, rather ...
Looks good. Oh, it looks like the binary [[ ]] shell operators slipped in there at some point. Those are bash only, I think, and should be fixed. But that's not part of this patch. -- Dan _______________________________________________ xorg mailing list xorg@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/xorg