[Buildroot] [PATCH] package/lmv2: add missing host-pkgconf dependency

[Romain Naour]

Hi Arnout,

Le 24/09/2017 à 15:01, Arnout Vandecappelle a écrit :
> 
> 
> On 13-09-17 22:30, Arnout Vandecappelle wrote:
>>
>>
>> On 13-09-17 22:17, Romain Naour wrote:
>>> configure.ac use PKG_CHECK_MODULES macro to find some libraries dependencies.
>>> So add host-pkgconf in LVM2_DEPENDENCIES.
>>>
>>> Fixes:
>>> checking pkg-config is at least version 0.9.0... ./configure: line 9875: [...]/host/bin/pkg-config: No such file or directory
>>
>>  Any idea why the autobuilders didn't hit this failure?
> 
>  Even though you didn't answer :-) I applied to master after all, since
> configure clearly really wants pkg-config. Thanks!

Sorry, I missed your reply :(

The autobuilders didn't hit this issue because when pkg-config is missing the
build doesn't stop.
I noticed it by chance while looking at a build log file.

Best regards,
Romain

> 
>  Regards,
>  Arnout
> 
>>>
>>> Signed-off-by: Romain Naour <romain.naour at gmail.com>
>>> ---
>>>  package/lvm2/lvm2.mk | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/package/lvm2/lvm2.mk b/package/lvm2/lvm2.mk
>>> index 63ac109..f7f7069 100644
>>> --- a/package/lvm2/lvm2.mk
>>> +++ b/package/lvm2/lvm2.mk
>>> @@ -21,6 +21,8 @@ LVM2_CONF_OPTS += \
>>>  	--enable-dmeventd \
>>>  	--disable-nls
>>>  
>>> +LVM2_DEPENDENCIES += host-pkgconf
>>> +
>>>  # LVM2 uses autoconf, but not automake, and the build system does not
>>>  # take into account the toolchain passed at configure time.
>>>  LVM2_MAKE_ENV = $(TARGET_CONFIGURE_OPTS)
>>>
>>
>