#3526 contrib/ci/deps.sh missing http-parser-devel dependency
Closed: Invalid Opened by akamenskiy.

Running $ ./contrib/ci/run on Centos6/7 minimal installation ends up in failure on configure step because ./contrib/ci/deps.sh does not attempt to install http-parser-devel on those systems.


So, EL6 is built without secrets support, thus http-parser-devel is not required.

For EL7, would you mind testing whether this quick patch works for you?

[ffidenci@pessoa sssd]$ git diff
diff --git a/contrib/ci/deps.sh b/contrib/ci/deps.sh
index 4467e117c..296318f93 100644
--- a/contrib/ci/deps.sh
+++ b/contrib/ci/deps.sh
@@ -50,6 +50,12 @@ if [[ "$DISTRO_BRANCH" == -redhat-* ]]; then
         krb5-server
         krb5-workstation
     )
+
+    if [[ "$DISTRO_BRANCH" != -redhat-redhatenterprise*-6.*- &&
+          "$DISTRO_BRANCH" != -redhat-centos-6.*- ]]; then
+        DEPS_LIST+=(http-parser-devel)
+    fi
+
     _DEPS_LIST_SPEC=`
         sed -e 's/@PACKAGE_VERSION@/0/g' \
             -e 's/@PACKAGE_NAME@/package-name/g' \

@fidencio Yes, this patch works

@akamenskiy: I've created a PR for this issue: https://github.com/SSSD/sssd/pull/396

Thanks for the report and testing!

Metadata Update from @fidencio:
- Issue assigned to fidencio

Metadata Update from @fidencio:
- Custom field patch adjusted to on

@akamenskiy We extract build dependencies from spec file and build dependency is listed there https://pagure.io/SSSD/sssd/blob/master/f/contrib/sssd.spec.in#_226

And as it is listed in other place you need at least el7.4 in case of rhel
https://pagure.io/SSSD/sssd/blob/master/f/contrib/sssd.spec.in#_111

Metadata Update from @lslebodn:
- Issue close_status updated to: Invalid
- Issue status updated to: Closed (was: Open)

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/4552

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata