<!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.32.2">
</HEAD>
<BODY>
On Wed, 2011-09-07 at 15:00 +0000, Egbert Eich wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
Sorry, forgot to metion in the header: this was for libXt.
</PRE>
</BLOCKQUOTE>
Can you also explain in the commit text the reasons why the location has changed? Most likely it is because the data it contains is architecture independent. Any impact regarding backward compatibility? Any thing a user should do after an upgrade?<BR>
<BR>
I don't know anything about libXt, just noticed the commit text being identical to the code.<BR>
<BR>
Why is datarootdir being added to the pc file? I don't see it being used in any way through pkg-config. Usually there is a separate variable for specific resources being queried by other modules. For example the server has sdkdir for the development header files. Should they ever change location, the callers using pkg-config would not be affected.<BR>
<BR>
This is the list of variables used by modules to query resources from other modules:
<BLOCKQUOTE>
<PRE>
--variable=appdefaultdir
--variable=dridriverdir
--variable=icondir
--variable=includedir
--variable=includex11dir
--variable=pythondir
--variable=sdkdir
--variable=sysconfigdir
--variable=systemdsystemunitdir
--variable=xcbincludedir
--variable=xkb_base
--variable=xthreadlib
</PRE>
</BLOCKQUOTE>
If there is a need for a module to get at the XtErrorDB path, I would suggest an explicit variable name. One should be able to figure it out simply by looking at the pc file.
<BLOCKQUOTE TYPE=CITE>
<PRE>
Cheers,
        Egbert.
On Wed, Sep 07, 2011 at 04:51:02PM +0200, Egbert Eich wrote:
> Signed-off-by: Egbert Eich <<A HREF="mailto:eich@freedesktop.org">eich@freedesktop.org</A>>
> ---
> src/Makefile.am | 2 +-
> xt.pc.in | 1 +
> 2 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 2b39ec5..727c077 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -4,7 +4,7 @@ BUILT_SOURCES = $(BUILT_FILES)
> CLEANFILES = $(BUILT_SOURCES)
>
> SRCH_DEFINES = -DXFILESEARCHPATHDEFAULT=\"$(XFILESEARCHPATHDEFAULT)\"
> -ERRORDB_DEFINES = -DERRORDB=\"$(libdir)/X11/XtErrorDB\"
> +ERRORDB_DEFINES = -DERRORDB=\"$(datarootdir)/X11/XtErrorDB\"
>
> AM_CFLAGS = $(CWARNFLAGS) $(XT_CFLAGS) $(SRCH_DEFINES) $(ERRORDB_DEFINES) \
>         $(XTMALLOC_ZERO_CFLAGS)
> diff --git a/xt.pc.in b/xt.pc.in
> index 28ce24a..755e070 100644
> --- a/xt.pc.in
> +++ b/xt.pc.in
> @@ -1,5 +1,6 @@
> prefix=@prefix@
> exec_prefix=@exec_prefix@
> +datarootdir=@datarootdir@
> libdir=@libdir@
> includedir=@includedir@
> appdefaultdir=@appdefaultdir@
> --
> 1.7.3.4
_______________________________________________
<A HREF="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</A>: X.Org development
Archives: <A HREF="http://lists.x.org/archives/xorg-devel">http://lists.x.org/archives/xorg-devel</A>
Info: <A HREF="http://lists.x.org/mailman/listinfo/xorg-devel">http://lists.x.org/mailman/listinfo/xorg-devel</A>
</PRE>
</BLOCKQUOTE>
<BR>
</BODY>
</HTML>