[PATCH wayland 2/2] build: Remove support for non-pkg-config Expat
Derek Foreman
derek.foreman.samsung at gmail.com
Wed Jul 25 17:38:27 UTC 2018
On 2018-07-21 06:13 AM, Daniel Stone wrote:
> The Expat XML library has shipped a pkg-config file for long enough to
> be in Debian's oldstable (Jessie, April 2015) and Ubuntu's oldest
> supported LTS (Trusty, 14.04). The pkg-config file was added in Expat
> upstream's commit 352cfc8f59a7, in September 2007.
>
> Drop build support for versions of Expat which do not ship a
> pkg-config file.
>
> Signed-off-by: Daniel Stone <daniels at collabora.com>
Looks good to me
Reviewed-by: Derek Foreman <derek.foreman.samsung at gmail.com>
> ---
> configure.ac | 11 +----------
> 1 file changed, 1 insertion(+), 10 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 8c2fb822..0022dcda 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -111,16 +111,7 @@ if test "x$enable_libraries" = "xyes"; then
> [[#include <time.h>]])
> fi
>
> -PKG_CHECK_MODULES(EXPAT, [expat], [],
> - [AC_CHECK_HEADERS(expat.h, [],
> - [AC_MSG_ERROR([Can't find expat.h. Please install expat.])])
> - SAVE_LIBS="$LIBS"
> - AC_SEARCH_LIBS(XML_ParserCreate, expat, [],
> - [AC_MSG_ERROR([Can't find expat library. Please install expat.])])
> - EXPAT_LIBS="$LIBS"
> - LIBS="$SAVE_LIBS"
> - AC_SUBST(EXPAT_LIBS)
> - ])
> +PKG_CHECK_MODULES(EXPAT, [expat])
>
> AM_CONDITIONAL([DTD_VALIDATION], [test "x$enable_dtd_validation" = "xyes"])
> if test "x$enable_dtd_validation" = "xyes"; then
>
More information about the wayland-devel
mailing list