Review Request 112448: FindSamba add PkgConfig fallback
Stephen Kelly
steveire at gmail.com
Mon Sep 9 19:48:55 UTC 2013
Christophe Giboudeaux wrote:
> On Monday 09 September 2013 21:37:37 Stephen Kelly wrote:
>> Stephen Kelly wrote:
>> > Christophe Giboudeaux wrote:
>> >> find_library(SAMBA_LIBRARIES NAMES smbclient HINTS ${PC_SAMBA_LIBDIR})
>> >
>> > This should be _LIBRARY, not _LIBRARIES. Followed by:
>> > set(SAMBA_LIBRARIES ${SAMBA_LIBRARY})
>>
>> The patch was committed, but this problem was not fixed.
>>
> On purpose, the fix went into 4.11 and master.
> I don't see the need to introduce _LIBRARY in these branches.
Interesting.
More information about the Kde-buildsystem
mailing list