]> xenbits.xensource.com Git - libvirt.git/commitdiff
wireshark: Use ${exec_prefix} instead of ${prefix}
authorAndrea Bolognani <abologna@redhat.com>
Wed, 26 Oct 2016 14:27:05 +0000 (16:27 +0200)
committerAndrea Bolognani <abologna@redhat.com>
Mon, 7 Nov 2016 09:16:06 +0000 (10:16 +0100)
${exec_prefix} and ${prefix} point to the same directory in
most setups, but when that's not the case the former should
be used for architecture-dependent data such as shared objects,
which makes it the best fit for our Wireshark dissector.

While at it, change all uses of $(var) to ${var}: they are
absolutely identicaly as far as make's concerned, but autoconf
itself seems to prefer the latter form so we might as well
follow suit.

m4/virt-wireshark.m4

index f253c86b23f5a2eb16282232dc498bc3febd9d7b..d5d7404382e0402af0bc646e42bb2134e6ea03d4 100644 (file)
@@ -29,23 +29,24 @@ AC_DEFUN([LIBVIRT_CHECK_WIRESHARK],[
   if test "x$with_wireshark_dissector" != "xno" ; then
     if test "x$with_ws_plugindir" = "xcheck" ; then
       ws_plugindir="$($PKG_CONFIG --variable plugindir wireshark)"
-      ws_prefix="$($PKG_CONFIG --variable prefix wireshark)"
+      ws_exec_prefix="$($PKG_CONFIG --variable exec_prefix wireshark)"
       ws_modversion="$($PKG_CONFIG --modversion wireshark)"
       if test "x$ws_plugindir" = "x" ; then
         dnl On some systems the plugindir variable may not be stored within pkg config.
         dnl Fall back to older style of constructing the plugin dir path.
         ws_plugindir="$libdir/wireshark/plugins/$ws_modversion"
       else
-        if test "x$ws_prefix" = "x" ; then
-          dnl If the wireshark prefix cannot be retrieved from pkg-config,
-          dnl /usr is our best bet
-          ws_prefix="/usr"
+        if test "x$ws_exec_prefix" = "x" ; then
+          dnl If wireshark's exec_prefix cannot be retrieved from pkg-config,
+          dnl this is our best bet
+          ws_exec_prefix="/usr"
         fi
-        dnl Replace the wireshark prefix with our own.
-        dnl Note that $(prefix) is kept verbatim at this point in time, and will
-        dnl only be expanded later, when make is called: this makes it possible
-        dnl to override the prefix at compilation or installation time
-        ws_plugindir='$(prefix)'"${ws_plugindir#$ws_prefix}"
+        dnl Replace wireshark's exec_prefix with our own.
+        dnl Note that ${exec_prefix} is kept verbatim at this point in time,
+        dnl and will only be expanded later, when make is called: this makes
+        dnl it possible to override such prefix at compilation or installation
+        dnl time
+        ws_plugindir='${exec_prefix}'"${ws_plugindir#$ws_exec_prefix}"
       fi
     elif test "x$with_ws_plugindir" = "xno" || test "x$with_ws_plugindir" = "xyes"; then
       AC_MSG_ERROR([ws-plugindir must be used only with valid path])