On Mon, Nov 23, 2015 at 09:18:47AM +0100, Jakub Hrozek wrote:
On Mon, Nov 23, 2015 at 09:10:59AM +0100, Lukas Slebodnik wrote:
> On (23/11/15 09:07), Jakub Hrozek wrote:
> >On Mon, Nov 23, 2015 at 08:09:29AM +0100, Lukas Slebodnik wrote:
> >> But it needn't be there. We do not have a strict requirement for
polkit.
> >> and Sumit mention that it's not required on rhel6. So it should not be
in
> >> buildroot.
> >>
> >> You might check for feature but we should not fail if the pkcheck file
> >> does not exist. Warning should be enough and configure script should
continue.
> >
> >So how would you propose to check if the directory exists?
> If binary pkcheck does not exist then the directory should not exist as well.
> Write warning with explanation that pk11_child might not work continue.
OK, I understand the proposal now, thanks.
Or at least I hope I understand, see the attached patch..pkexec is still
checkef for in the buildroot, but if it's not there, the directory check
is silently skipped.
CI passed with this patch:
http://sssd-ci.duckdns.org/logs/job/33/80/summary.html