Hello community,

here is the log from the commit of package gssproxy for openSUSE:Factory 
checked in at 2019-07-08 15:10:51
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/gssproxy (Old)
 and      /work/SRC/openSUSE:Factory/.gssproxy.new.4615 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "gssproxy"

Mon Jul  8 15:10:51 2019 rev:3 rq:713631 version:0.8.2

Changes:
--------
--- /work/SRC/openSUSE:Factory/gssproxy/gssproxy.changes        2019-07-02 
10:38:51.770624453 +0200
+++ /work/SRC/openSUSE:Factory/.gssproxy.new.4615/gssproxy.changes      
2019-07-08 15:10:54.727338588 +0200
@@ -1,0 +2,10 @@
+Thu Jul  4 14:33:52 UTC 2019 - Samuel Cabrero <[email protected]>
+
+- Fix paths in tests and replace python's f-string usage
+- Test are passing in OBS, drop patch disable-failing-test.patch
+- Added patches:
+  * 0001-Fix-runtests.py.patch
+- Removed patches:
+  * disable-failing-test.patch
+
+-------------------------------------------------------------------

Old:
----
  disable-failing-test.patch

New:
----
  0001-Fix-runtests.py.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ gssproxy.spec ++++++
--- /var/tmp/diff_new_pack.xaT4GU/_old  2019-07-08 15:10:55.411339244 +0200
+++ /var/tmp/diff_new_pack.xaT4GU/_new  2019-07-08 15:10:55.411339244 +0200
@@ -25,7 +25,7 @@
 URL:            https://pagure.io/gssproxy
 Source0:        https://releases.pagure.org/gssproxy/%{name}-%{version}.tar.gz
 # PATCH-FIX-SUSE [email protected] disable test that fails only on OBS builds
-Patch0:         disable-failing-test.patch
+Patch0:         0001-Fix-runtests.py.patch
 BuildRequires:  docbook-xsl-stylesheets
 BuildRequires:  doxygen
 BuildRequires:  krb5-client
@@ -34,6 +34,8 @@
 BuildRequires:  openldap2
 BuildRequires:  openldap2-client
 BuildRequires:  pkgconfig
+BuildRequires:  python3
+BuildRequires:  system-user-nobody
 BuildRequires:  systemd-rpm-macros
 BuildRequires:  valgrind
 BuildRequires:  pkgconfig(ini_config) >= 1.2.0
@@ -64,15 +66,6 @@
 %prep
 %setup -q
 %patch0 -p1
-# add more paths for krb binaries in tests
-sed \
-    -e 
's#/sbin:/bin:%{_prefix}/sbin:%{_prefix}/bin#/sbin:/bin:%{_prefix}/sbin:%{_prefix}/bin:%{_prefix}/lib/mit/sbin#'
 \
-    -i tests/runtests.py \
-    -i tests/testlib.py
-# fix path for krb5 ldap schema
-sed -i \
-    -e 's#%{_datadir}/doc/krb5-server-ldap\*/#%{_datadir}/kerberos/ldap/#g' \
-    tests/testlib.py
 
 %build
 autoreconf -fvi

++++++ 0001-Fix-runtests.py.patch ++++++
>From ac7b0aa70ae285252e910fda5a1ef475d7e31dd1 Mon Sep 17 00:00:00 2001
From: Samuel Cabrero <[email protected]>
Date: Tue, 2 Jul 2019 12:19:00 +0200
Subject: [PATCH] Fix runtests.py

* f-strings require python3.6
* Add slapd to PATH
* kerberos ldap schema path

Signed-off-by: Samuel Cabrero <[email protected]>
---
 tests/runtests.py | 4 ++--
 tests/testlib.py  | 9 +++++----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/tests/runtests.py b/tests/runtests.py
index 9cd546f..5b7d85e 100755
--- a/tests/runtests.py
+++ b/tests/runtests.py
@@ -12,10 +12,10 @@ import testlib
 from testlib import *
 
 def check_exec(name):
-    env = {'PATH': '/sbin:/bin:/usr/sbin:/usr/bin'}
+    env = {'PATH': 
'/sbin:/bin:/usr/sbin:/usr/bin:/usr/lib/mit/bin:/usr/lib/mit/sbin:/usr/lib/openldap'}
     ret = subprocess.call(["which", name], stdout=subprocess.DEVNULL, env=env)
     if ret != 0:
-        print(f"Executable '{name}' not found in {env['PATH']}",
+        print("Executable '%s' not found in %s" % (name, env['PATH']),
               file=sys.stderr)
         exit(1)
 
diff --git a/tests/testlib.py b/tests/testlib.py
index 05c734d..8d02e5d 100755
--- a/tests/testlib.py
+++ b/tests/testlib.py
@@ -300,8 +300,9 @@ def write_ldap_krb5_config(testdir):
         raise ValueError("Did not find LDAP schemas; is openldap installed?")
 
     k5schema = None
-    for path in ["/usr/share/doc/krb5-server-ldap*/kerberos.schema",
-                 "/usr/share/doc/krb5-kdc-ldap/kerberos.schema.gz"]:
+    # Path in TW and SLE-15-SP1 onward /usr/share/kerberos/ldap
+    for path in ["/usr/share/kerberos/ldap/kerberos.schema",
+                 "/usr/share/doc/packages/krb5/kerberos.schema"]:
         pathlist = glob.glob(path)
         if len(pathlist) > 0:
             k5schema = pathlist[0]
@@ -367,7 +368,7 @@ def setup_ldap(testdir, wrapenv):
     stashfile = os.path.join(testdir, "ldap_passwd")
     krb5conf = os.path.join(testdir, 'krb5.conf')
 
-    ldapenv = {'PATH': '/sbin:/bin:/usr/sbin:/usr/bin',
+    ldapenv = {'PATH': 
'/sbin:/bin:/usr/sbin:/usr/bin:/usr/lib/openldap:/usr/lib/mit/sbin',
                'KRB5_CONFIG': krb5conf}
     ldapenv.update(wrapenv)
 
@@ -411,7 +412,7 @@ def setup_kdc(testdir, wrapenv):
     kdcstash = os.path.join(kdcdir, KDC_STASH)
     kdcdb = os.path.join(kdcdir, KDC_DBNAME)
 
-    kdcenv = {'PATH': '/sbin:/bin:/usr/sbin:/usr/bin',
+    kdcenv = {'PATH': 
'/sbin:/bin:/usr/sbin:/usr/bin:/usr/lib/mit/sbin:/usr/lib/mit/bin',
               'KRB5_CONFIG': krb5conf,
               'KRB5_KDC_PROFILE': kdcconf}
     kdcenv.update(wrapenv)
-- 
2.22.0


Reply via email to