URL:
https://github.com/SSSD/sssd/pull/995
Title: #995: nss: Collision with external nss symbol
mzidek-gh commented:
"""
@sgallagher If we want to make the naming consistent and remove the nss_ then I would
prefer to do it in another PR and keep this one as a minimal change to unblock rawhide.
There are couple of things that are not clear to me for now:
- do we want to remove nss_ from the entire codebase or just from this one file (for the
latter case the final result would not look that much consistent, the former looks quite
big), or are we talking about some specific subset of functions we want to rename?
- is it worth the future conflicts when backporting patches to older branches? (it
probably is)
I am not against the change you propose, but I would prefer to postpone the above
discussion and move it to a ticket and push this smaller version now.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/995#issuecomment-592067037