PR#4: Rewrite has_creds -

cccolutils

#4 Rewrite has_creds

Merged Dec 12, 2016 by puiterwijk. Opened Dec 1, 2016 by cqi.

cqi/cccolutils rewrite-has_creds into master

Chenxiong Qi • Dec 1, 2016

c5e3c6a36532a33560dbb28949932e2ca0480123

cqi commented Dec 1, 2016

Original has_creds calls krb5_cccol_have_content that is only available since
version 1.11 of krb5 library, but in EL6, e.g. the CentOS and RHEL6, just an
old version in the repository. Rewriting has_creds aims to make cccolutils be
usable in all Fedora dist and both EL6 and EL7.

Signed-off-by: Chenxiong Qi cqi@redhat.com

cqi commented Dec 1, 2016

This fixes https://bodhi.fedoraproject.org/updates/python-cccolutils-1.4-1.el6#comment-528802

cqi commented Dec 6, 2016

@puiterwijk Hi, could you review? The new release requires this patch so that python-cccolutils is workable in EL6.

Edited Dec 6, 2016 by cqi

lsedlar commented Dec 6, 2016

The package with this patch builds fine (ugly scratch build). It allows cccolutils to be imported correctly on EL6 and has_creds correctly detects that I have no credentials in the docker container I used for testing.

Pull-Request has been merged by puiterwijk

Dec 12, 2016