Re: [PATCH xorg-gtest] Remove xorg-server pkgconfig check

2012-02-29 Thread Chase Douglas

On 02/29/2012 03:15 PM, Gaetan Nadon wrote:

On 12-02-29 03:28 PM, Chase Douglas wrote:

xorg-server flags are not needed. Including them also causes
-fvisibility=hidden to be used, which breaks the build output.

Signed-off-by: Chase Douglas
---
  configure.ac |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/configure.ac b/configure.ac
index ae254be..89a72a7 100644
--- a/configure.ac
+++ b/configure.ac
@@ -32,7 +32,7 @@ XORG_DEFAULT_OPTIONS
  AC_PROG_CXX
  AC_PROG_INSTALL

-PKG_CHECK_MODULES( XSERVER, x11 [xorg-server] )
+PKG_CHECK_MODULES(XSERVER, x11)

  # Check for Google Test
  AC_CHECK_GTEST

That will do it. The name XSERVER is very confusing, but that can be
changed later.

Reviewed-by: Gaetan Nadon



And pushed.

Thanks!
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel


Re: [PATCH xorg-gtest] Remove xorg-server pkgconfig check

2012-02-29 Thread Gaetan Nadon
On 12-02-29 03:28 PM, Chase Douglas wrote:
> xorg-server flags are not needed. Including them also causes
> -fvisibility=hidden to be used, which breaks the build output.
>
> Signed-off-by: Chase Douglas 
> ---
>  configure.ac |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index ae254be..89a72a7 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -32,7 +32,7 @@ XORG_DEFAULT_OPTIONS
>  AC_PROG_CXX
>  AC_PROG_INSTALL
>  
> -PKG_CHECK_MODULES( XSERVER, x11 [xorg-server] )
> +PKG_CHECK_MODULES(XSERVER, x11)
>   
>  # Check for Google Test
>  AC_CHECK_GTEST
That will do it. The name XSERVER is very confusing, but that can be
changed later.

Reviewed-by: Gaetan Nadon
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel