Hello community, here is the log from the commit of package rpmlint for openSUSE:Factory checked in at 2016-09-09 10:15:40 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/rpmlint (Old) and /work/SRC/openSUSE:Factory/.rpmlint.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "rpmlint" Changes: -------- --- /work/SRC/openSUSE:Factory/rpmlint/rpmlint.changes 2016-09-01 14:01:39.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.rpmlint.new/rpmlint.changes 2016-09-09 10:15:49.000000000 +0200 @@ -1,0 +2,12 @@ +Tue Sep 6 14:48:55 UTC 2016 - lars@linux-schulserver.de + +- Adding sks (Synchronizing Key Server) to known user/group + +------------------------------------------------------------------- +Tue Sep 06 13:29:48 UTC 2016 - lnussel@suse.de + +- rpmlint-checks: + * Make checks python3 compatible + * Fix update-alternatives check when it's quoted + +------------------------------------------------------------------- ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ rpmlint.spec: same change ++++++ _servicedata ++++++ --- /var/tmp/diff_new_pack.Mt78hy/_old 2016-09-09 10:15:51.000000000 +0200 +++ /var/tmp/diff_new_pack.Mt78hy/_new 2016-09-09 10:15:51.000000000 +0200 @@ -3,4 +3,4 @@ <param name="url">http://github.com/openSUSE/rpmlint-tests.git</param> <param name="changesrevision">8d99488dee290853c470199fa1fe80aec8e33784</param></service><service name="tar_scm"> <param name="url">http://github.com/openSUSE/rpmlint-checks.git</param> - <param name="changesrevision">08a7268fa35a0a043199bb3f32172c26e69f1ad3</param></service></servicedata> \ No newline at end of file + <param name="changesrevision">5e8508e132800b5f1afb6abd4ce7003c9e0c248a</param></service></servicedata> \ No newline at end of file ++++++ config ++++++ --- /var/tmp/diff_new_pack.Mt78hy/_old 2016-09-09 10:15:51.000000000 +0200 +++ /var/tmp/diff_new_pack.Mt78hy/_new 2016-09-09 10:15:51.000000000 +0200 @@ -210,6 +210,7 @@ 'shadow', 'shibd', 'siproxd', + 'sks', 'snort', 'squid', 'sshd', @@ -375,6 +376,7 @@ 'sensu', 'shibd', 'siproxd', + 'sks', 'snort', 'squid', 'sshd', ++++++ rpmlint-checks-master.tar.xz ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/rpmlint-checks-master/BashismsCheck.py new/rpmlint-checks-master/BashismsCheck.py --- old/rpmlint-checks-master/BashismsCheck.py 2015-05-20 10:45:11.000000000 +0200 +++ new/rpmlint-checks-master/BashismsCheck.py 2016-09-06 15:28:50.000000000 +0200 @@ -34,6 +34,8 @@ printInfo(pkg, "potential-bashisms", filename) except Exception as x: printError(pkg, 'rpmlint-exception', "%(file)s raised an exception: %(x)s" % {'file':filename, 'x':x}) + except UnicodeDecodeError: + pass finally: f.close() diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/rpmlint-checks-master/BrandingPolicyCheck.py new/rpmlint-checks-master/BrandingPolicyCheck.py --- old/rpmlint-checks-master/BrandingPolicyCheck.py 2016-07-25 16:27:42.000000000 +0200 +++ new/rpmlint-checks-master/BrandingPolicyCheck.py 2016-09-06 15:28:50.000000000 +0200 @@ -22,7 +22,7 @@ if pkg.isSource(): return - pkg_requires = set(map(lambda x: string.split(x[0], '(')[0], pkg.requires())) + pkg_requires = set(map(lambda x: str.split(x[0], '(')[0], pkg.requires())) pkg_conflicts = set(map(lambda x: x[0], pkg.conflicts())) # verify that only generic branding is required by non-branding packages @@ -52,7 +52,7 @@ branding_style = pkg.name.partition('-branding-')[2] generic_branding = ("%s-branding" % (branding_basename)) - pkg_provides = set(map(lambda x: string.split(x[0], '(')[0], pkg.provides())) + pkg_provides = set(map(lambda x: str.split(x[0], '(')[0], pkg.provides())) pkg_supplements = set(map(lambda x: x[0], pkg.supplements())) # verify that it only supplements with packageand diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/rpmlint-checks-master/CheckFilelist.py new/rpmlint-checks-master/CheckFilelist.py --- old/rpmlint-checks-master/CheckFilelist.py 2015-05-20 10:45:11.000000000 +0200 +++ new/rpmlint-checks-master/CheckFilelist.py 2016-09-06 15:28:50.000000000 +0200 @@ -397,7 +397,7 @@ invalidopt = set() isSUSE = (pkg.header[RPMTAG_VENDOR] and - 'SUSE' in pkg.header[RPMTAG_VENDOR]) + b'SUSE' in pkg.header[RPMTAG_VENDOR]) # the checks here only warn about a directory once rather # than reporting potentially hundreds of files individually diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/rpmlint-checks-master/CheckInitScripts.py new/rpmlint-checks-master/CheckInitScripts.py --- old/rpmlint-checks-master/CheckInitScripts.py 2015-05-20 10:45:11.000000000 +0200 +++ new/rpmlint-checks-master/CheckInitScripts.py 2016-09-06 15:28:50.000000000 +0200 @@ -25,8 +25,8 @@ return files = pkg.files() - bins_list = filter(lambda f: (f.startswith("/usr/bin") \ - or f.startswith("/usr/sbin")) and stat.S_ISREG(files[f].mode), files.keys()) + bins_list = (f for f in files.keys() if (f.startswith("/usr/bin") \ + or f.startswith("/usr/sbin")) and stat.S_ISREG(files[f].mode)) for f, pkgfile in files.items(): @@ -35,7 +35,7 @@ boot_script = f.startswith('/etc/init.d/boot.') - input_f = file(pkg.dirName() + '/' + f, "r") + input_f = open(pkg.dirName() + '/' + f, "r") found_remote_fs = False for l in input_f: if l.startswith('# Required-Start') or l.startswith('# Should-Start'): @@ -65,7 +65,7 @@ if dep == '4': printError(pkg, "init-script-runlevel-4", f) - if not found_remote_fs and bins_list: + if not found_remote_fs and any(bins_list): printWarning(pkg, "non-remote_fs-dependency", f) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/rpmlint-checks-master/CheckKDE4Deps.py new/rpmlint-checks-master/CheckKDE4Deps.py --- old/rpmlint-checks-master/CheckKDE4Deps.py 2015-05-20 10:45:11.000000000 +0200 +++ new/rpmlint-checks-master/CheckKDE4Deps.py 2016-09-06 15:28:50.000000000 +0200 @@ -61,7 +61,7 @@ if pkg.isSource(): return - pkg_requires = set(map(lambda x: string.split(x[0],'(')[0], pkg.requires())) + pkg_requires = set(map(lambda x: str.split(x[0],'(')[0], pkg.requires())) if not "libkdecore.so.5" in pkg_requires: return diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/rpmlint-checks-master/CheckPkgConfig.py new/rpmlint-checks-master/CheckPkgConfig.py --- old/rpmlint-checks-master/CheckPkgConfig.py 2015-05-20 10:45:11.000000000 +0200 +++ new/rpmlint-checks-master/CheckPkgConfig.py 2016-09-06 15:28:50.000000000 +0200 @@ -37,7 +37,7 @@ if pkg.grep(self.suspicious_dir, filename): Filter.printError(pkg, "invalid-pkgconfig-file", filename) - pc_file = file(pkg.dirName() + "/" + filename, "r") + pc_file = open(pkg.dirName() + "/" + filename, "r") for l in pc_file: if l.startswith('Libs:') and self.wronglib_dir.search(l): Filter.printError(pkg, 'pkgconfig-invalid-libs-dir', diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/rpmlint-checks-master/CheckUpdateAlternatives.py new/rpmlint-checks-master/CheckUpdateAlternatives.py --- old/rpmlint-checks-master/CheckUpdateAlternatives.py 2015-05-20 10:45:11.000000000 +0200 +++ new/rpmlint-checks-master/CheckUpdateAlternatives.py 2016-09-06 15:28:50.000000000 +0200 @@ -31,13 +31,13 @@ @classmethod def read_ghost_files(cls, script): - if not script or not 'update-alternatives' in script: + if not script or not b'update-alternatives' in script: return set() ghost_files = set() for command in ( - c.replace('\\\n', '').strip() + c.replace('\\\n', '').replace('"','').strip() for c in script.split('update-alternatives') if cls.INSTALL in c): diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/rpmlint-checks-master/DuplicatesCheck.py new/rpmlint-checks-master/DuplicatesCheck.py --- old/rpmlint-checks-master/DuplicatesCheck.py 2015-05-20 10:45:11.000000000 +0200 +++ new/rpmlint-checks-master/DuplicatesCheck.py 2016-09-06 15:28:50.000000000 +0200 @@ -15,7 +15,7 @@ def get_prefix(file): - pathlist = string.split(file, '/') + pathlist = str.split(file, '/') if len(pathlist) == 3: return "/".join(pathlist[0:2]) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/rpmlint-checks-master/LibraryPolicyCheck.py new/rpmlint-checks-master/LibraryPolicyCheck.py --- old/rpmlint-checks-master/LibraryPolicyCheck.py 2015-05-20 10:45:11.000000000 +0200 +++ new/rpmlint-checks-master/LibraryPolicyCheck.py 2016-09-06 15:28:50.000000000 +0200 @@ -284,7 +284,7 @@ def libname_from_soname(soname): - libname = string.split(soname, '.so.') + libname = str.split(soname, '.so.') if len(libname) == 2: if libname[0][-1:].isdigit(): libname = string.join(libname, '-') @@ -319,7 +319,7 @@ libs_to_dir = dict() dirs = set() reqlibs = set() - pkg_requires = set(map(lambda x: string.split(x[0], '(')[0], + pkg_requires = set(map(lambda x: str.split(x[0], '(')[0], pkg.requires())) for f, pkgfile in files.items(): @@ -432,7 +432,7 @@ done = set() for dir in dirs: if dir.startswith(sysdir + '/'): - ssdir = string.split(dir[len(sysdir)+1:], '/')[0] + ssdir = str.split(dir[len(sysdir)+1:], '/')[0] if not ssdir[-1].isdigit(): cdirs.add(sysdir+'/'+ssdir) done.add(dir)