BUILD: Drop libsss_config

[fidencio]

libssss_config has been used only by OpenLMI and the project has been
deprecated making, then, no sense to keep the support on SSSD.

Distros that, for some reason, are still packing and distributing
OpenLMI can stick to SSSD 1.14 branch.

Signed-off-by: Fabiano Fidêncio fidencio@redhat.com

[fidencio]

CI: http://sssd-ci.duckdns.org/logs/job/56/93/summary.html

[lslebodn]

You forgot to rebuve build dependencies from spec file and from ci.deps contrib/ci/deps.sh: libaugeas-dev contrib/sssd.spec.in:BuildRequires: augeas-devel LS

[fidencio]

Change done. Here is the fix up patch that in order to help the reviewer:

[ffidenci@cat sssd]$ git diff HEAD
diff --git a/contrib/ci/deps.sh b/contrib/ci/deps.sh
index 9a7098c..387ad1f 100644
--- a/contrib/ci/deps.sh
+++ b/contrib/ci/deps.sh
@@ -69,7 +69,6 @@ if [[ "$DISTRO_BRANCH" == -debian-* ]]; then
         docbook-xsl
         gettext
         krb5-config
-        libaugeas-dev
         libc-ares-dev
         libcmocka-dev
         libcollection-dev
diff --git a/contrib/sssd.spec.in b/contrib/sssd.spec.in
index 2917629..f0b61ec 100644
--- a/contrib/sssd.spec.in
+++ b/contrib/sssd.spec.in
@@ -583,7 +583,6 @@ be used by Python applications.
 Summary: The D-Bus responder of the SSSD
 Group: Applications/System
 License: GPLv3+
-BuildRequires: augeas-devel
 Requires: sssd-common = %{version}-%{release}

 %description dbus

[lslebodn]

Is there a reason why ifp_component_change_debug_level_tmp was not removed?

[lslebodn]

There is a tiny conflict due to change in src/external/configlib.m4

There is also typo in commit message libssss_config and IMHO it might be better to use "BUILD"
as a component in commit title rather then LIBSSS_CONFIG.

I think you also forgot to remove methods from src/responder/ifp/ifp_iface.xml which is used
for generating C code.

[fidencio]

Suggestion taken and changes done, thanks for the review!
I've just pushed a new version of the patch.

[lslebodn]

You probably missed one comment. Is there a reason why ifp_component_change_debug_level_tmp was not removed?

Otherwise LGTM

[lslebodn]

Ahh, I've just noticed that this function does not depend on auges/libsss_config.
It just changes values in confdb. Which is temporary just by a chance and it would work only after restarting services. Do we need to keep it?

@pbrezina What do you think about remoing ChangeDebugLevelTemporarily?

[lslebodn]

ChangeDebugLevelTemporarily does not depend on auges so it can be removed later.
master:

• 99b2352

[lslebodn]

Just let you know
the openlmi is orphaned and was removed from fedora