Fwd: Undocumented variable ${X11_X11_LIB} in FindX11.cmake

[Alexander Neundorf]

On Sunday 15 November 2009, José Manuel Santamaría Lema wrote:
> Hi, I'm the one who send the original message to kde-devel ml,
>
> > On Saturday 14 November 2009, Fathi Boudra wrote:
> > > forward to the right mailing list.
> > >
> > > ---------- Forwarded message ----------
> > > Hi,
> > >
> > > I asked already in IRC if it's safe to use this variable:
> > > ${X11_X11_LIB} from FindX11.cmake[1]; someone answered yes, which
> > > sounds logical.
> >
> > In general, as long as it is undocumented, you're on your own if you use
> > it. But for  it really looks like it has been forgotten to
> > document.
>
> I Agree, note that afaik there is no way to link against libX11.la without
> linking to the whole libX*.la ... unless you use X11_X11_LIB.
> For instance, see what I did in kmplayer:
> http://websvn.kde.org/?view=revision&revision=1046843
> I could use X11_LIBRARIES, but, afaik I would be linking uselessly to the
> whole libX*.la.
>
> > > So, could somebody with more kdelibs/cmake knowledge than me please
> > > confirm that it's safe to use ${X11_X11_LIB} (I think so) and also
> > > review the currently documented variables and add the undocumented
> > > ones? (I'm afraid that there are various variables which sould be
> > > documented)
> >
> > Which ones ?
>
> The ones related with X11_X11_LIB; I *guess* that X11_Xlib_INCLUDE_PATH and
> X11_X11_INCLUDE_PATH should be documented too.

Already done, both in kdelibs/cmake/modules/ as well as cmake.

Alex