diff options
| author | Paul Eggert <eggert@cs.ucla.edu> | 2012-04-07 13:45:36 -0700 |
|---|---|---|
| committer | Paul Eggert <eggert@cs.ucla.edu> | 2012-04-07 13:45:36 -0700 |
| commit | bf6bba2b03fb576b46f904887dc1f2963d230902 (patch) | |
| tree | 877122498db03fa98f5d41d5cee753cfc3aba407 | |
| parent | dfae128a8a03835816213a37ef44a30e1bdd44cd (diff) | |
| download | emacs-bf6bba2b03fb576b46f904887dc1f2963d230902.tar.gz | |
Check pkg-config exit status when configuring (Bug#10626).
* configure.in (PKG_CHECK_MODULES): Do not assume that pkg-config
works; check its exit status. Problem reported by Jordi Gutiérrez
Hermoso in
<http://lists.gnu.org/archive/html/emacs-devel/2012-01/msg00787.html>.
| -rw-r--r-- | ChangeLog | 8 | ||||
| -rw-r--r-- | configure.in | 17 |
2 files changed, 15 insertions, 10 deletions
diff --git a/ChangeLog b/ChangeLog index c3903258200..fa6a34220c0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2012-04-07 Paul Eggert <eggert@cs.ucla.edu> + + Check pkg-config exit status when configuring (Bug#10626). + * configure.in (PKG_CHECK_MODULES): Do not assume that pkg-config + works; check its exit status. Problem reported by Jordi Gutiérrez + Hermoso in + <http://lists.gnu.org/archive/html/emacs-devel/2012-01/msg00787.html>. + 2012-04-07 Glenn Morris <rgm@gnu.org> * README, configure.in (AC_INIT): Bump version to 24.1.50. diff --git a/configure.in b/configure.in index 20b6de07246..7babf879128 100644 --- a/configure.in +++ b/configure.in @@ -1124,17 +1124,14 @@ AC_DEFUN([PKG_CHECK_MODULES], [ if $PKG_CONFIG --atleast-pkgconfig-version $PKG_CONFIG_MIN_VERSION; then AC_MSG_CHECKING(for $2) - if $PKG_CONFIG --exists "$2" 2>&AS_MESSAGE_LOG_FD; then - AC_MSG_RESULT(yes) - succeeded=yes - - AC_MSG_CHECKING($1_CFLAGS) - $1_CFLAGS=`$PKG_CONFIG --cflags "$2"|sed -e 's,///*,/,g'` - AC_MSG_RESULT($$1_CFLAGS) + if $PKG_CONFIG --exists "$2" 2>&AS_MESSAGE_LOG_FD && + $1_CFLAGS=`$PKG_CONFIG --cflags "$2" 2>&AS_MESSAGE_LOG_FD` && + $1_LIBS=`$PKG_CONFIG --libs "$2" 2>&AS_MESSAGE_LOG_FD`; then - AC_MSG_CHECKING($1_LIBS) - $1_LIBS=`$PKG_CONFIG --libs "$2"|sed -e 's,///*,/,g'` - AC_MSG_RESULT($$1_LIBS) + $1_CFLAGS=`AS_ECHO(["$$1_CFLAGS"]) | sed -e 's,///*,/,g'` + $1_LIBS=`AS_ECHO(["$$1_LIBS"]) | sed -e 's,///*,/,g'` + AC_MSG_RESULT([yes CFLAGS='$$1_CFLAGS' LIBS='$$1_LIBS']) + succeeded=yes else AC_MSG_RESULT(no) $1_CFLAGS="" |