D6000: Handle differences in PAM headers

2017-05-31 Thread Adriaan de Groot
This revision was automatically updated to reflect the committed changes. Closed by commit R107:38ff660bb81c: Handle differences in PAM headers (authored by adridg). REPOSITORY R107 KWallet PAM Integration CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6000?vs=14912&id=15009 REVISIO

D6000: Handle differences in PAM headers

2017-05-31 Thread Jonathan Riddell
jriddell accepted this revision. jriddell added a comment. This revision is now accepted and ready to land. doesn't break the build on Neon REPOSITORY R107 KWallet PAM Integration BRANCH master REVISION DETAIL https://phabricator.kde.org/D6000 To: adridg, jriddell, #plasma Cc: plasma-

D6000: Handle differences in PAM headers

2017-05-28 Thread Adriaan de Groot
adridg added a comment. See, e.g., https://build-sandbox.kde.org/view/Plasma/job/Plasma kwallet-pam kf5-qt5 FreeBSDQt5.7/1/console That's the compile failure (pam_kwallet.c:39:10: fatal error: 'security/pam_ext.h' file not found) without these changes. REPOSITORY R107 KWallet PAM Inte

D6000: Handle differences in PAM headers

2017-05-28 Thread Adriaan de Groot
adridg added reviewers: jriddell, Plasma. REPOSITORY R107 KWallet PAM Integration REVISION DETAIL https://phabricator.kde.org/D6000 To: adridg, jriddell, #plasma Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D6000: Handle differences in PAM headers

2017-05-28 Thread Adriaan de Groot
adridg created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY FreeBSD has neither pam_ext.h nor pam_syslog(), so check for headers and add defines as appropriate. TEST PLAN Builds on FreeBSD, now. Need