All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] fix an error message when running configure
@ 2012-03-22  1:40 Mikulas Patocka
  2012-03-22  3:33 ` [PATCH] integrate veritysetup to lvm Mikulas Patocka
  0 siblings, 1 reply; 6+ messages in thread
From: Mikulas Patocka @ 2012-03-22  1:40 UTC (permalink / raw)
  To: lvm-devel

Fix an error reported by configure

This code path calls "$PKG_CONFIG" without initializing the PKG_CONFIG
variable. It results in this error when running ./configure:

"./configure: line 9757: --variable=systemdsystemunitdir: command not found"

The patch changes it so that we try to find pkg_config and initialize
$PKG_CONFIG. If pkg-config isn't found and the variable is still empty,
we don't try to run "$PKG_CONFIG"

Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>

---
 configure.in |    8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

Index: LVM2.2.02.95/configure.in
===================================================================
--- LVM2.2.02.95.orig/configure.in	2012-03-21 20:02:36.000000000 +0100
+++ LVM2.2.02.95/configure.in	2012-03-21 20:17:38.000000000 +0100
@@ -1208,7 +1208,13 @@ AC_ARG_WITH(systemdsystemunitdir,
 	    AC_HELP_STRING([--with-systemdsystemunitdir=DIR],
 			   [systemd service files in DIR]),
 	    systemdsystemunitdir=$withval,
-	    pkg_systemdsystemunitdir=$($PKG_CONFIG --variable=systemdsystemunitdir systemd))
+	    if test x$PKGCONFIG_INIT != x1; then
+		pkg_config_init
+	    fi
+	    if test -n "$PKG_CONFIG"; then
+		pkg_systemdsystemunitdir=$($PKG_CONFIG --variable=systemdsystemunitdir systemd)
+	    fi
+	    )
 
 if test -n "$pkg_systemdsystemunitdir"; then
 	    systemdsystemunitdir=$pkg_systemdsystemunitdir;



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2012-03-24  2:09 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-22  1:40 [PATCH] fix an error message when running configure Mikulas Patocka
2012-03-22  3:33 ` [PATCH] integrate veritysetup to lvm Mikulas Patocka
2012-03-23 11:04   ` Zdenek Kabelac
2012-03-23 11:38     ` Alasdair G Kergon
2012-03-23 19:12     ` Mikulas Patocka
     [not found]   ` <20120324004159.GP16214@agk-dp.fab.redhat.com>
     [not found]     ` <Pine.LNX.4.64.1203232058110.3024@file.rdu.redhat.com>
     [not found]       ` <20120324010220.GQ16214@agk-dp.fab.redhat.com>
     [not found]         ` <Pine.LNX.4.64.1203232108100.3024@file.rdu.redhat.com>
2012-03-24  2:09           ` Mikulas Patocka

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.