On Mon, Jan 11, 2016 at 02:51:24PM +0100, Lukas Slebodnik wrote:
On (11/01/16 12:53), Jakub Hrozek wrote:
>On Mon, Jan 11, 2016 at 12:22:19PM +0100, Lukas Slebodnik wrote:
>> On (11/01/16 10:26), Jakub Hrozek wrote:
>> >On Wed, Jan 06, 2016 at 04:48:00PM +0100, Lukas Slebodnik wrote:
>> >> ehlo,
>> >> This change is required to reduce dependency tree in client
>> >> container (or on atomic host, ...)
>> >> Patch is attached.
>> >> LS
>> >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
>> >the filesystem?
>> It was in sssd-common but it should be moved outside.
>> I dnifn't notice I forgot to do that :-)
>Ah, sorry, I meant sssd-client here, not sssd-common...since client
>would have to be installed anyway in the containerized environments and
>the size of the sudo package is just 20kBs..
I think it's explained in commit message
or should I ellaborate?
I must admit I didn't read it and yes, it's clear now, thanks.
I'll proceed to patch review..