<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
<META NAME="GENERATOR" CONTENT="GtkHTML/3.26.0">
</HEAD>
<BODY>
On Sun, 2010-08-08 at 09:13 -0700, Alan Coopersmith wrote:<BR>
<BLOCKQUOTE TYPE=CITE>
<TT><FONT COLOR="#1a1a1a">While it might make sense to add an xkbbell option to specify the</FONT></TT><BR>
<TT><FONT COLOR="#1a1a1a">bell name from libxkbfile's XKBbells.h, that's not done now, so</FONT></TT><BR>
<TT><FONT COLOR="#1a1a1a">the dependency can be removed. (It would also make sense if</FONT></TT><BR>
<TT><FONT COLOR="#1a1a1a">XKBbells.h was not part of libxkbfile, which appears to be just</FONT></TT><BR>
<TT><FONT COLOR="#1a1a1a">"an undocumented private library we can hide this in until we</FONT></TT><BR>
<TT><FONT COLOR="#1a1a1a"> decide if it's ready for libX11" - a number of the other X apps</FONT></TT><BR>
<TT><FONT COLOR="#1a1a1a">do have conditional code to use it, some of which got broken in</FONT></TT><BR>
<TT><FONT COLOR="#1a1a1a">the imake -> autoconf conversion.)</FONT></TT><BR>
<BR>
</BLOCKQUOTE>
I had a look, it's broken in some modules, done in different ways in other modules.<BR>
<BR>
I'd rather proceed with this patch to leave the code in a consistent state.<BR>
Whatever the future is, the statement would not remain as is.<BR>
<BR>
Thanks
</BODY>
</HTML>