[libvirt] [PATCH 3/3] wireshark: Use ${exec_prefix} for $ws_plugindir
Viktor Mihajlovski
mihajlov at linux.vnet.ibm.com
Thu Oct 27 08:35:34 UTC 2016
On 26.10.2016 18:08, Andrea Bolognani wrote:
> ${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 from $(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 | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/m4/virt-wireshark.m4 b/m4/virt-wireshark.m4
> index 75786de..7a817f6 100644
> --- a/m4/virt-wireshark.m4
> +++ b/m4/virt-wireshark.m4
> @@ -52,10 +52,10 @@ AC_DEFUN([LIBVIRT_CHECK_WIRESHARK],[
> ws_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 Note that ${exec_prefix} is kept verbatim at this point in time, and
> + dnl will only be expanded later, when make is called: this makes it
> + dnl possible to override such prefix at compilation or installation time
> + ws_plugindir='${exec_prefix}'"${ws_plugindir#$ws_prefix}"
> 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])
> else
>
Purely cosmetic, but s/ws_prefix/ws_exec_prefix/ in the entire file
could avoid future confusions.
--
Mit freundlichen Grüßen/Kind Regards
Viktor Mihajlovski
IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martina Köderitz
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294
More information about the libvir-list
mailing list