[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] configure: change a typo in zstd config
From: |
Juan Quintela |
Subject: |
Re: [PATCH] configure: change a typo in zstd config |
Date: |
Thu, 05 Mar 2020 10:31:43 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Philippe Mathieu-Daudé <address@hidden> wrote:
> On 3/3/20 1:56 PM, Juan Quintela wrote:
>> Denis Plotnikov <address@hidden> wrote:
>>> Package manager --exist flag is used instead of --exists.
>>> Fix it.
>>>
>>> Signed-off-by: Denis Plotnikov <address@hidden>
>>
>> Reviewed-by: Juan Quintela <address@hidden>
>>
>> queued.
>
> It doesn't seem that trivial:
> https://www.mail-archive.com/address@hidden/msg684754.html
I think you need to do a make clean.
And I found why I didn't catch that problem. Fedora pkg-config is soooo
intelligent:
$ pkg-config --exist libzstd; echo $?
0
$ pkg-config --exists libzstd; echo $?
0
$ pkg-config --exists libzstd33; echo $?
1
$ pkg-config --exist libzstd33; echo $?
1
$
I tried with libzstd-devel installed and uninstalled, with --enable-zstd
and --disable-zstd, and it always did the right thing for me.
Later, Juan.
>
>>
>> thanks.
>>
>>> ---
>>> configure | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/configure b/configure
>>> index 7b373bc0bb..caa65f5883 100755
>>> --- a/configure
>>> +++ b/configure
>>> @@ -2464,7 +2464,7 @@ fi
>>> # zstd check
>>> if test "$zstd" != "no" ; then
>>> - if $pkg_config --exist libzstd ; then
>>> + if $pkg_config --exists libzstd ; then
>>> zstd_cflags="$($pkg_config --cflags libzstd)"
>>> zstd_libs="$($pkg_config --libs libzstd)"
>>> LIBS="$zstd_libs $LIBS"
>>
>>