update license list, fix desktop file parsing
This commit is contained in:
parent
cd043ba906
commit
2be746cdc8
31
rpmlint-1.5-desktopfix.patch
Normal file
31
rpmlint-1.5-desktopfix.patch
Normal file
@ -0,0 +1,31 @@
|
|||||||
|
diff -up rpmlint-1.5/MenuXDGCheck.py.desktopfix rpmlint-1.5/MenuXDGCheck.py
|
||||||
|
--- rpmlint-1.5/MenuXDGCheck.py.desktopfix 2013-10-09 19:33:21.067330420 -0400
|
||||||
|
+++ rpmlint-1.5/MenuXDGCheck.py 2013-10-09 19:33:50.422330861 -0400
|
||||||
|
@@ -41,14 +41,19 @@ class MenuXDGCheck(AbstractCheck.Abstrac
|
||||||
|
|
||||||
|
self.cfp.read(f)
|
||||||
|
binary = self.cfp.get('Desktop Entry','Exec').split(' ',1)[0]
|
||||||
|
- 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 not found and binary:
|
||||||
|
- printWarning(pkg, 'desktopfile-without-binary', filename, binary)
|
||||||
|
+ if binary:
|
||||||
|
+ if binary.startswith('/'):
|
||||||
|
+ found = os.path.exists(root + binary)
|
||||||
|
+ else:
|
||||||
|
+ 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
|
||||||
|
+ break
|
||||||
|
+ if not found:
|
||||||
|
+ printWarning(pkg, 'desktopfile-without-binary', filename,
|
||||||
|
+ binary)
|
||||||
|
|
||||||
|
check = MenuXDGCheck()
|
||||||
|
|
@ -40,7 +40,7 @@ setOption("DanglingSymlinkExceptions", (
|
|||||||
setOption("ValidLicenses", (
|
setOption("ValidLicenses", (
|
||||||
# These are the short names for all of the Fedora approved licenses.
|
# These are the short names for all of the Fedora approved licenses.
|
||||||
# The master list is kept here: http://fedoraproject.org/wiki/Licensing
|
# The master list is kept here: http://fedoraproject.org/wiki/Licensing
|
||||||
# Last synced with revision "2.13, 8 May 2013" of that page.
|
# Last synced with revision "2.17, 31 July 2013" of that page.
|
||||||
'AAL',
|
'AAL',
|
||||||
'Abstyles',
|
'Abstyles',
|
||||||
'Adobe',
|
'Adobe',
|
||||||
@ -49,6 +49,7 @@ setOption("ValidLicenses", (
|
|||||||
'Afmparse',
|
'Afmparse',
|
||||||
'AGPLv1',
|
'AGPLv1',
|
||||||
'AGPLv3',
|
'AGPLv3',
|
||||||
|
'AGPLv3+',
|
||||||
'AGPLv3 with exceptions',
|
'AGPLv3 with exceptions',
|
||||||
'AMDPLPA',
|
'AMDPLPA',
|
||||||
'AML',
|
'AML',
|
||||||
@ -92,6 +93,7 @@ setOption("ValidLicenses", (
|
|||||||
'Crystal Stacker',
|
'Crystal Stacker',
|
||||||
'Cube',
|
'Cube',
|
||||||
'diffmark',
|
'diffmark',
|
||||||
|
'DMIT',
|
||||||
'DOC',
|
'DOC',
|
||||||
'Dotseqn',
|
'Dotseqn',
|
||||||
'DSDP',
|
'DSDP',
|
||||||
@ -161,6 +163,7 @@ setOption("ValidLicenses", (
|
|||||||
'libtiff',
|
'libtiff',
|
||||||
'LLGPL',
|
'LLGPL',
|
||||||
'Logica',
|
'Logica',
|
||||||
|
'LOSLA',
|
||||||
'LPL',
|
'LPL',
|
||||||
'LPPL',
|
'LPPL',
|
||||||
'MakeIndex',
|
'MakeIndex',
|
||||||
@ -241,6 +244,7 @@ setOption("ValidLicenses", (
|
|||||||
'Teeworlds',
|
'Teeworlds',
|
||||||
'Threeparttable',
|
'Threeparttable',
|
||||||
'TMate',
|
'TMate',
|
||||||
|
'Tolua',
|
||||||
'TORQUEv1.1',
|
'TORQUEv1.1',
|
||||||
'TOSL',
|
'TOSL',
|
||||||
'TPL',
|
'TPL',
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
Name: rpmlint
|
Name: rpmlint
|
||||||
Version: 1.5
|
Version: 1.5
|
||||||
Release: 3%{?dist}
|
Release: 4%{?dist}
|
||||||
Summary: Tool for checking common errors in RPM packages
|
Summary: Tool for checking common errors in RPM packages
|
||||||
|
|
||||||
Group: Development/Tools
|
Group: Development/Tools
|
||||||
@ -14,6 +14,8 @@ Source3: %{name}-etc.config
|
|||||||
Source4: %{name}.config.el4
|
Source4: %{name}.config.el4
|
||||||
# EL-5 specific config
|
# EL-5 specific config
|
||||||
Source5: %{name}.config.el5
|
Source5: %{name}.config.el5
|
||||||
|
# http://sourceforge.net/p/rpmlint/code/ci/f85186d/
|
||||||
|
Patch0: rpmlint-1.5-desktopfix.patch
|
||||||
|
|
||||||
BuildArch: noarch
|
BuildArch: noarch
|
||||||
BuildRequires: python >= 2.4
|
BuildRequires: python >= 2.4
|
||||||
@ -49,6 +51,7 @@ and source packages as well as spec files can be checked.
|
|||||||
|
|
||||||
%prep
|
%prep
|
||||||
%setup -q
|
%setup -q
|
||||||
|
%patch0 -p1 -b .desktopfix
|
||||||
sed -i -e /MenuCheck/d Config.py
|
sed -i -e /MenuCheck/d Config.py
|
||||||
cp -p config config.example
|
cp -p config config.example
|
||||||
install -pm 644 %{SOURCE2} CHANGES.package.old
|
install -pm 644 %{SOURCE2} CHANGES.package.old
|
||||||
@ -99,6 +102,10 @@ make check
|
|||||||
|
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Wed Oct 9 2013 Tom Callaway <spot@fedoraproject.org> - 1.5-4
|
||||||
|
- Fix handling of Exec= with an absolute path (bz991278)
|
||||||
|
- Update license list, add AGPLv3+ (bz894187)
|
||||||
|
|
||||||
* Tue Aug 6 2013 Thomas Woerner <twoerner@redhat.com> - 1.5-3
|
* Tue Aug 6 2013 Thomas Woerner <twoerner@redhat.com> - 1.5-3
|
||||||
- Fixed URL and Source0, now using sourceforge.net
|
- Fixed URL and Source0, now using sourceforge.net
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user