[SSSD] [sssd PR#502][comment] BUILD: Add missing libs found by -Wl,-z,defs

2018-02-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/502 Title: #502: BUILD: Add missing libs found by -Wl,-z,defs lslebodn commented: """ master: * b61304a9f0f6bcc099cb855bc5bde13c4024da1a * 7ad9f9c7c8b40e74647de455e454894a257bc0de """ See the full comment at

[SSSD] [sssd PR#502][comment] BUILD: Add missing libs found by -Wl,-z,defs

2018-02-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/502 Title: #502: BUILD: Add missing libs found by -Wl,-z,defs jhrozek commented: """ No, go ahead and push the patches. I don't pretend I checked all additions of all symbols, but I spot checked a few and the changes look good. And the proxy link with

[SSSD] [sssd PR#502][comment] BUILD: Add missing libs found by -Wl,-z,defs

2018-02-05 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/502 Title: #502: BUILD: Add missing libs found by -Wl,-z,defs lslebodn commented: """ BTW there is also downstream bug https://bugzilla.redhat.com/show_bug.cgi?id=1538997 Should I split this patch to few small patches? because it would be good to fix fix

[SSSD] [sssd PR#502][comment] BUILD: Add missing libs found by -Wl,-z,defs

2018-01-25 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/502 Title: #502: BUILD: Add missing libs found by -Wl,-z,defs lslebodn commented: """ On (23/01/18 12:05), Jakub Hrozek wrote: >How should I test this? I tried to revert (err, comment out) the sed that >removes the -z flag from linker flags, then apply

[SSSD] [sssd PR#502][comment] BUILD: Add missing libs found by -Wl,-z,defs

2018-01-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/502 Title: #502: BUILD: Add missing libs found by -Wl,-z,defs jhrozek commented: """ btw here is the specfile diff: ``` @@ -31,7 +31,7 @@ Name: sssd Version: 1.16.0 -Release: 7%{?dist} +Release: 7%{?dist}.2 Group: Applications/System Summary: System

[SSSD] [sssd PR#502][comment] BUILD: Add missing libs found by -Wl,-z,defs

2018-01-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/502 Title: #502: BUILD: Add missing libs found by -Wl,-z,defs jhrozek commented: """ How should I test this? I tried to revert (err, comment out) the sed that removes the -z flag from linker flags, then apply the two patches, but the fedora build on