Accepting request 186070 from devel:openSUSE:Factory:rpmlint

- add xdg-check-exception.diff:
  handle xdg menu files without exec entry (bnc#833384)

OBS-URL: https://build.opensuse.org/request/show/186070
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/rpmlint?expand=0&rev=195
This commit is contained in:
Stephan Kulow 2013-08-06 10:03:56 +00:00 committed by Git OBS Bridge
commit 8400b362a9
3 changed files with 35 additions and 0 deletions

View File

@ -1,3 +1,9 @@
-------------------------------------------------------------------
Tue Aug 6 09:21:04 UTC 2013 - dmueller@suse.com
- add xdg-check-exception.diff:
handle xdg menu files without exec entry (bnc#833384)
------------------------------------------------------------------- -------------------------------------------------------------------
Wed Jul 31 11:42:54 UTC 2013 - meissner@suse.com Wed Jul 31 11:42:54 UTC 2013 - meissner@suse.com

View File

@ -122,6 +122,7 @@ Patch89: suse-python-abi-check.diff
Patch91: suse-manpages-for-rc-scripts.patch Patch91: suse-manpages-for-rc-scripts.patch
# PATCH-FIX-UPSTREAM: lnussel@suse.de - Fix setgroups error name # PATCH-FIX-UPSTREAM: lnussel@suse.de - Fix setgroups error name
Patch92: rpmlint-1.5-Fix-setgroups-error-name.diff Patch92: rpmlint-1.5-Fix-setgroups-error-name.diff
Patch93: xdg-check-exception.diff
%py_requires %py_requires
@ -198,6 +199,7 @@ source packages can be checked.
%patch89 %patch89
%patch91 -p1 %patch91 -p1
%patch92 -p1 %patch92 -p1
%patch93 -p1
cp -p %{SOURCE2} . cp -p %{SOURCE2} .
# Only move top-level python files # Only move top-level python files
chmod 0755 rpmlint-checks-master/*.py chmod 0755 rpmlint-checks-master/*.py

27
xdg-check-exception.diff Normal file
View File

@ -0,0 +1,27 @@
diff --git a/MenuXDGCheck.py b/MenuXDGCheck.py
index 0996585..29085d5 100644
--- a/MenuXDGCheck.py
+++ b/MenuXDGCheck.py
@@ -40,13 +40,16 @@ class MenuXDGCheck(AbstractCheck.AbstractFilesCheck):
printError(pkg, 'non-utf8-desktopfile', filename)
self.cfp.read(f)
- binary = self.cfp.get('Desktop Entry','Exec').split(' ',1)[0]
+ binary = None
found = False
- for i in STANDARD_BIN_DIRS:
- if os.path.exists(root + i + binary):
- # no need to check if the binary is +x, rpmlint does it
- # in another place
- found = True
+ if self.cfp.has_option('Desktop Entry','Exec'):
+ binary = self.cfp.get('Desktop Entry','Exec').split(' ',1)[0]
+
+ for i in STANDARD_BIN_DIRS:
+ if os.path.exists(root + i + binary):
+ # no need to check if the binary is +x, rpmlint does it
+ # in another place
+ found = True
if not found and binary:
printWarning(pkg, 'desktopfile-without-binary', filename, binary)