AC_CHECK_PROG checks for program in given path. However, if it doesn't
exists, [variable] is set to [value-if-not-found]. We don't want this
to be the empty string in case of 'modprobe' and 'scrub' as we want to
fallback to runtime detection.
---
configure.ac | 4 ++--
src/util/pci.c | 5 -----
2 files changed, 2 insertions(+), 7 deletions(-)
diff --git a/configure.ac b/configure.ac
index 4493b94..732f4fe 100644
--- a/configure.ac
+++ b/configure.ac
@@ -211,11 +211,11 @@ AC_PATH_PROG([UDEVADM], [udevadm], [],
[/sbin:/usr/sbin:/usr/local/sbin:$PATH])
AC_PATH_PROG([UDEVSETTLE], [udevsettle], [],
[/sbin:/usr/sbin:/usr/local/sbin:$PATH])
-AC_PATH_PROG([MODPROBE], [modprobe], [],
+AC_PATH_PROG([MODPROBE], [modprobe], [modprobe],
[/sbin:/usr/sbin:/usr/local/sbin:$PATH])
AC_PATH_PROG([OVSVSCTL], [ovs-vsctl], [ovs-vsctl],
[/sbin:/usr/sbin:/usr/local/sbin:$PATH])
-AC_PATH_PROG([SCRUB], [scrub], [],
+AC_PATH_PROG([SCRUB], [scrub], [scrub],
[/sbin:/usr/sbin:/usr/local/sbin:$PATH])
AC_DEFINE_UNQUOTED([DNSMASQ],["$DNSMASQ"],
diff --git a/src/util/pci.c b/src/util/pci.c
index 633dcd8..c660e8d 100644
--- a/src/util/pci.c
+++ b/src/util/pci.c
@@ -40,11 +40,6 @@
#include "virterror_internal.h"
#include "virfile.h"
-/* avoid compilation breakage on some systems */
-#ifndef MODPROBE
-# define MODPROBE "modprobe"
-#endif
-
#define PCI_SYSFS "/sys/bus/pci/"
#define PCI_ID_LEN 10 /* "XXXX XXXX" */
#define PCI_ADDR_LEN 13 /* "XXXX:XX:XX.X" */
--
1.7.3.4
Show replies by date
On Wed, Feb 22, 2012 at 11:28:33 +0100, Michal Privoznik wrote:
AC_CHECK_PROG checks for program in given path. However, if it
doesn't
exists, [variable] is set to [value-if-not-found]. We don't want this
to be the empty string in case of 'modprobe' and 'scrub' as we want to
fallback to runtime detection.
---
configure.ac | 4 ++--
src/util/pci.c | 5 -----
2 files changed, 2 insertions(+), 7 deletions(-)
ACK
Jirka
On 22.02.2012 12:09, Jiri Denemark wrote:
On Wed, Feb 22, 2012 at 11:28:33 +0100, Michal Privoznik wrote:
> AC_CHECK_PROG checks for program in given path. However, if it doesn't
> exists, [variable] is set to [value-if-not-found]. We don't want this
> to be the empty string in case of 'modprobe' and 'scrub' as we want
to
> fallback to runtime detection.
> ---
> configure.ac | 4 ++--
> src/util/pci.c | 5 -----
> 2 files changed, 2 insertions(+), 7 deletions(-)
ACK
Jirka
Thanks, pushed.
Michal