forked from pool/rpmlint
Dirk Mueller
1d8d6856a8
* Avoid exception on inaccessible scripts * Print out the error content on UnicodeError to make flake8 happy * Fix flake8 warning about missing space around operators * Use compressions when checking for backup files * Account for arch specific code in /usr/share * Check for installed libtool wrapper files * Check for missing optional dependencies * Consider gnome help for doc files * Check for noarch package with files in lib64 * Verify if description is longer than summary * Explicitly tell users how to set URL * Ignore pytest_cache directory * confusing-invalid-spec-name * Ignore orig/rej leftovers after patching * Reenable Travis testing against Fedora Rawhide * Check all sections that should not use %buildroot in them * Put in default buildroot value used by Fedora/openSUSE * Stricter interpreter check * Use compileall to avoid %buildroot to be in pyc * Drop deprecated config file usage, 0.88 is pretty old anyway * Adjust Version to not print outdated Copyright * Rework Travis checks against latest Centos and Fedora releases * Fix exception handling * Fix various flake8-import-order test regressions * Blacklist newer pycodestyle warnings * Fix compatibility with file 5.33+ * Python 3.7.0b5 magic number is 3394 * Update TagsCheck.py * pyc related tests: DRY OBS-URL: https://build.opensuse.org/package/show/devel:openSUSE:Factory:rpmlint/rpmlint?expand=0&rev=648
60 lines
2.4 KiB
Diff
60 lines
2.4 KiB
Diff
From: Some One <nobody@opensuse.org>
|
|
Date: Thu, 9 Apr 2015 14:55:38 +0200
|
|
Subject: [PATCH] suse-filter-more-verbose.diff
|
|
|
|
===================================================================
|
|
---
|
|
Config.py | 25 +++++++++++++++++++++++--
|
|
1 file changed, 23 insertions(+), 2 deletions(-)
|
|
|
|
Index: rpmlint-rpmlint-1.11/Config.py
|
|
===================================================================
|
|
--- rpmlint-rpmlint-1.11.orig/Config.py
|
|
+++ rpmlint-rpmlint-1.11/Config.py
|
|
@@ -10,6 +10,7 @@
|
|
import locale
|
|
import os.path
|
|
import re
|
|
+import sys
|
|
|
|
try:
|
|
from __version__ import __version__
|
|
@@ -177,7 +178,17 @@ def isFiltered(s):
|
|
if '(' in _filters_non_except[idx]:
|
|
_non_named_group_re.subn('(:?', _filters_non_except[idx])
|
|
_filters_non_except_re = _filters_non_except_re + '|(?:' + _filters_non_except[idx] +')'
|
|
- _filters_non_except_re = re.compile(_filters_non_except_re)
|
|
+ try:
|
|
+ _filters_non_except_re = re.compile(_filters_non_except_re)
|
|
+ except Exception:
|
|
+ # Try to figure out which filter caused the error
|
|
+ for f in _filters_non_except[1:]:
|
|
+ try:
|
|
+ re.compile(f)
|
|
+ except Exception as e:
|
|
+ # can't use Pkg.error/warn here, as that would recurse
|
|
+ print >> sys.stderr, "(none): E: %s in filter '%s'" % (e.message, f)
|
|
+ sys.exit(2)
|
|
|
|
if _filters_re == None and len(_filters):
|
|
_filters_re = '(?:' + _filters[0] + ')'
|
|
@@ -189,7 +200,17 @@ def isFiltered(s):
|
|
if '(' in _filters[idx]:
|
|
_non_named_group_re.subn('(:?', _filters[idx])
|
|
_filters_re = _filters_re + '|(?:' + _filters[idx] + ')'
|
|
- _filters_re = re.compile(_filters_re)
|
|
+ try:
|
|
+ _filters_re = re.compile(_filters_re)
|
|
+ except Exception:
|
|
+ # Try to figure out which filter caused the error
|
|
+ for f in _filters[1:]:
|
|
+ try:
|
|
+ re.compile(f)
|
|
+ except Exception as e:
|
|
+ # can't use Pkg.error/warn here, as that would recurse
|
|
+ print >> sys.stderr, "(none): E: %s in filter '%s'" % (e.message, f)
|
|
+ sys.exit(2)
|
|
|
|
if _filters_except_re == None and len(_filters_except):
|
|
_filters_except_re = '(?:' + _filters_except[0] + ')'
|