Respect '--without-infopipe' in 'sssctl' and tests

[alexey-tikhonov]

No description provided.

[pbrezina]

Thank you. I wonder if it wouldn't be better to remove the build condition? Infopipe is there for years and built by default, I think the build condition was introduced for times when ifp was experimental.

[alexey-tikhonov]

if it wouldn't be better to remove the build condition?

Minimalistic container service doesn't need 'sssd_ifp' (and probably doesn't run dbus-broker).

[pbrezina]

if it wouldn't be better to remove the build condition?

Minimalistic container service doesn't need 'sssd_ifp' (and probably doesn't run dbus-broker).

In this scenario however, sssd will still be installed via packaging system and not built directly on the container. So ifp will be built, only sssd-dbus package won't be installed. Or do I miss something?

[alexey-tikhonov]

if it wouldn't be better to remove the build condition?

Minimalistic container service doesn't need 'sssd_ifp' (and probably doesn't run dbus-broker).

In this scenario however, sssd will still be installed via packaging system and not built directly on the container. So ifp will be built, only sssd-dbus package won't be installed. Or do I miss something?

I'm using this build option in #7262 to experiment with resulting copr build.

One of the things I'm trying to do there is to minimize install dependencies to be able to install SSSD on a really minimal container image.
So in that PR I switched off everything not really needed.

Probably '--without-infopipe' indeed isn't important (libdbus dependency is there due to SBUS anyway, so maybe there are no extra deps), but I thought having this option fixed won't hurt...

[pbrezina]

if it wouldn't be better to remove the build condition?

Minimalistic container service doesn't need 'sssd_ifp' (and probably doesn't run dbus-broker).

In this scenario however, sssd will still be installed via packaging system and not built directly on the container. So ifp will be built, only sssd-dbus package won't be installed. Or do I miss something?

I'm using this build option in #7262 to experiment with resulting copr build.

One of the things I'm trying to do there is to minimize install dependencies to be able to install SSSD on a really minimal container image. So in that PR I switched off everything not really needed.

Probably '--without-infopipe' indeed isn't important (libdbus dependency is there due to SBUS anyway, so maybe there are no extra deps), but I thought having this option fixed won't hurt...

It shouldn't bring any extra deps. Won't hurt, I'd personally like to see it removed instead of complicate things since I don't think there is any value in the conditional build now. But I can go either way, the patch is fine, obviously. However we would need to also add proper skip to system tests.

[alexey-tikhonov]

Ok, I'll remove build option instead.

[alexey-tikhonov]

Ok, I'll remove build option instead.

#7545