On (11/01/16 10:26), Jakub Hrozek wrote:
On Wed, Jan 06, 2016 at 04:48:00PM +0100, Lukas Slebodnik wrote:
> This change is required to reduce dependency tree in client
> container (or on atomic host, ...)
> Patch is attached.
I have some questions..
1) Why not just move the file ownership to sssd-common? Are you
concerned that not everyone needs sudo and it would be an extra file on
It was in sssd-common but it should be moved outside.
I dnifn't notice I forgot to do that :-)
2) The src/sss_client/sudo/sss_sudo.c only lists GPLv3, not LGPLv3,
you sure adding the lesser copying is correct?
I didn't check header of file. I used the same license as in was in 1.9.x
3) I think this is a good opportunity to move the library outside
libdir, at least in F-24 (this would require coordination with sudo and
adding versioned Requires)
They are unrelated changes and shoudl be discussed with
BTW atm sssd-common(and thus libsss_sudo.so) does not depend on sudo.
Updated patch is attached.