URL: https://github.com/SSSD/sssd/pull/109 Title: #109: SSSCTL: fix netgroup-show parsing
lslebodn commented: """ On (07/01/17 08:28), Jakub Hrozek wrote: > >> On 6 Jan 2017, at 16:29, lslebodn <[email protected]> wrote: >> >> On (06/01/17 05:52), Jakub Hrozek wrote: >> >This patch is OK, but only for sssd-1-14. In master, we already fall back >> >to parsing the name string as short name if parsing the qualified name >> >fails. I’m not thrilled about that, because it can conceal legitimate >> >errors, but it’s needed atm to use cache_req everywhere. >> > >> >So yeah, ack to this patch for sssd-1-14 >> > >> Do you think that it would be a problem to backport >> 7b293a5095ef3e63cd2e3f2ff01b7484bf6dcd38 into 1.14 >> rather that this patch? >> >> Upstream integration tests passed with it. I haven't tried downstrem >> tests. >> > >That would also work. > OK It was fixed in master by different commit and not by this PR. It was a dependency for refactoring of nsss responder. And it's better to have the same commit in older branches. master: * 7b293a5095ef3e63cd2e3f2ff01b7484bf6dcd38 sssd-1-14: * c61f9fe1781fc4da2fc9e7f11071a4827cb0e463 LS """ See the full comment at https://github.com/SSSD/sssd/pull/109#issuecomment-271513113
_______________________________________________ sssd-devel mailing list -- [email protected] To unsubscribe send an email to [email protected]
