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