On 10/18/21 22:12, Richard Biener wrote:
On Mon, 18 Oct 2021, Thorsten Kukuk wrote:
Hi,
On Mon, Oct 18, Michael Ströder wrote:
On 10/18/21 08:56, Dan Čermák wrote:
Hi Michael,
Michael Ströder
writes: HI!
Building package nss-pam-ldapd for Leap and SLE currently fails:
https://build.opensuse.org/package/show/network:ldap/nss-pam-ldapd
RPM build errors: File must begin with "/": %{_pam_moduledir}/pam_ldap.so
Probably %{_pam_moduledir} is wrong for these platforms.
it looks like %_pam_moduledir is not defined on Leap & SLE: [..] So you'll probably have to revert to the good'ol hackery: --8<---------------cut here---------------start------------->8--- %{!?_pam_moduledir: %define _pam_moduledir %{_libdir}/security} --8<---------------cut here---------------end--------------->8---
Thank you! Submitted this work-around.
Which is wrong for Leap 15.3/SLE15 SP3 and older.
It should be: %{!?_pam_moduledir: %define _pam_moduledir /%{_lib}/security}
If _pam_moduledir is not set, usrMerge did most likely not happen and the PAM modules are still in /lib{64}/security and not /usr/lib{64}/security
Btw, I do think it would make maintainance and Factory-first _much_ easier if we at least would try to introduce forward compatibility in old codestreams when new macros like this appear in Factory. Not to say that it would bogus workarounds to creep in.
There are many cases where we have done such, its probably just a matter of people not being aware that they can do so. At the same time it does require building from the backports projects rather then just the SLE ones because only the backports one generally pulls in stuff from update repos. -- Simon Lees (Simotek) http://simotek.net Emergency Update Team keybase.io/simotek SUSE Linux Adelaide Australia, UTC+10:30 GPG Fingerprint: 5B87 DB9D 88DC F606 E489 CEC5 0922 C246 02F0 014B