[PATCH libXtst] Added missing "xi" requires to pkconfig file (#48585)

Alan Coopersmith alan.coopersmith at oracle.com
Tue Jul 2 20:26:27 PDT 2013


Isn't having it in the Requires.private enough for that, since it's just
a *.h dependency, not on the *.so?

Seems like this should have been covered by:
http://cgit.freedesktop.org/xorg/lib/libXtst/commit/?id=6394218aae6599b28ced3d30074e154ba4218d07
but we forgot to close the bugzilla bug then  (would have helped if I
included the link to the bug instead of just to the patch attachment).

	-alan-

On 07/ 2/13 08:16 PM, Peter Hutterer wrote:
> From: Stefan Dirsch <sndirsch at suse.de>
>
> xi is an external dependency, XTest.h exposes it.
>
> X.Org Bug 48585 <http://bugs.freedesktop.org/show_bug.cgi?id=48585>
>
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
>   xtst.pc.in | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/xtst.pc.in b/xtst.pc.in
> index 5426416..208bfe1 100644
> --- a/xtst.pc.in
> +++ b/xtst.pc.in
> @@ -6,7 +6,7 @@ includedir=@includedir@
>   Name: Xtst
>   Description: The Xtst Library
>   Version: @PACKAGE_VERSION@
> -Requires: recordproto
> -Requires.private: x11 xext xextproto xi
> +Requires: recordproto xi
> +Requires.private: x11 xext xextproto
>   Cflags: -I${includedir}
>   Libs: -L${libdir} -lXtst
>


-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list