Bug 888385 - Review Request: python-yapsy - Fat-free DIY Python plugin management toolkit
Summary: Review Request: python-yapsy - Fat-free DIY Python plugin management toolkit
Keywords:
Status: CLOSED DUPLICATE of bug 998738
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mario Blättermann
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-12-18 15:51 UTC by José Matos
Modified: 2013-09-09 21:33 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-09 14:57:50 UTC
Type: ---
Embargoed:
mario.blaettermann: fedora-review?


Attachments (Terms of Use)

Description José Matos 2012-12-18 15:51:46 UTC
Spec URL: http://jamatos.fedorapeople.org/python-yapsy.spec
SRPM URL: http://jamatos.fedorapeople.org/python-yapsy-1.9.2-1.fc18.src.rpm
Description:
A simple framework for plugin system development with as few
dependencies as possible. It is designed to offer a set of very lean
classes (plugin managers and plugin interfaces) which can easily be
customised by decoration or inheritance.


Fedora Account System Username: jamatos

Comment 1 José Matos 2012-12-18 16:35:37 UTC
New version that requires at build time both python-setuptools (v2 and v3).

Spec URL: http://jamatos.fedorapeople.org/python-yapsy.spec
SRPM URL: http://jamatos.fedorapeople.org/python-yapsy-1.9.2-2.fc18.src.rpm

Comment 2 Yohan Graterol 2013-01-22 05:21:22 UTC
Hello Jose Matos, I'm not officially packager, but I want do a informal review at your package. :)

there are a issue in the test with rpmlint to file .rpm, here is the output

$ rpmlint python-yapsy-1.9.2-2.fc19.noarch.rpm 
python-yapsy.noarch: W: spelling-error %description -l en_US customised -> customized, customize, accustomed
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/AutoInstallPluginManager.py 0644L /usr/bin/python
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/PluginInfo.py 0644L /usr/bin/python
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/PluginManager.py 0644L /usr/bin/python
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/ConfigurablePluginManager.py 0644L /usr/bin/python
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/IPlugin.py 0644L /usr/bin/python
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/PluginManagerDecorator.py 0644L /usr/bin/python
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/FilteredPluginManager.py 0644L /usr/bin/python
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/VersionedPluginManager.py 0644L /usr/bin/python
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/__init__.py 0644L /usr/bin/python
1 packages and 0 specfiles checked; 9 errors, 1 warnings.

Good luck!

Comment 3 Mario Blättermann 2013-06-02 14:55:35 UTC
The tarball contains an upstream egg, you have to delete it before building the package so that it gets rebuilt:

%prep
%setup -q -n Yapsy-%{version}-pythons2n3
rm -rf *.egg-info

See http://fedoraproject.org/wiki/Packaging:Python_Eggs#Upstream_Eggs for more info.

Don't mix %{buildroot} and $RPM_BUILD_ROOT in the same spec file.


$ rpmlint -i -v *
python-yapsy.src: I: checking
python-yapsy.src: W: spelling-error %description -l en_US customised -> customized, customize, accustomed
The value of this tag appears to be misspelled. Please double-check.

Please fix the spelling error:
customised → customized

Comment 4 José Matos 2013-06-18 14:30:53 UTC
(In reply to Mario Blättermann from comment #3)
> The tarball contains an upstream egg, you have to delete it before building
> the package so that it gets rebuilt:
> 
> %prep
> %setup -q -n Yapsy-%{version}-pythons2n3
> rm -rf *.egg-info
> 
> See http://fedoraproject.org/wiki/Packaging:Python_Eggs#Upstream_Eggs for
> more info.

I noticed it the first time but forgot to remove it. Mea culpa. :-)

> Don't mix %{buildroot} and $RPM_BUILD_ROOT in the same spec file.

I was surprised when I noticed the $ version in there. :-)

> $ rpmlint -i -v *
> python-yapsy.src: I: checking
> python-yapsy.src: W: spelling-error %description -l en_US customised ->
> customized, customize, accustomed
> The value of this tag appears to be misspelled. Please double-check.
> 
> Please fix the spelling error:
> customised → customized

Done.

I have updated the package to the latest stable version together with all the fixes you proposed.

Spec URL: http://jamatos.fedorapeople.org/python-yapsy.spec
SRPM URL: http://jamatos.fedorapeople.org/python-yapsy-1.10.2-1.fc20.src.rpm

Comment 5 Mario Blättermann 2013-06-22 08:00:49 UTC
Scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=5530158

$ rpmlint -i -v *
python3-yapsy.noarch: I: checking
python3-yapsy.noarch: I: checking-url http://yapsy.sourceforge.net/ (timeout 10 seconds)
python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/PluginInfo.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/ConfigurablePluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/IPlugin.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/VersionedPluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/__init__.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/PluginManagerDecorator.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/PluginFileLocator.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/AutoInstallPluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/PluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/FilteredPluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python3-yapsy.noarch: E: non-executable-script /usr/lib/python3.3/site-packages/yapsy/IPluginLocator.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: I: checking
python-yapsy.noarch: I: checking-url http://yapsy.sourceforge.net/ (timeout 10 seconds)
python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/AutoInstallPluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/IPluginLocator.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/PluginInfo.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/PluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/ConfigurablePluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/IPlugin.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/PluginManagerDecorator.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/FilteredPluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/VersionedPluginManager.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/__init__.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/yapsy/PluginFileLocator.py 0644L /usr/bin/python
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

python-yapsy.src: I: checking
python-yapsy.src: I: checking-url http://yapsy.sourceforge.net/ (timeout 10 seconds)
python-yapsy.src: I: checking-url http://downloads.sourceforge.net/yapsy/Yapsy-1.10.2/Yapsy-1.10.2-pythons2n3.tar.gz (timeout 10 seconds)
python-yapsy.spec: I: checking-url http://downloads.sourceforge.net/yapsy/Yapsy-1.10.2/Yapsy-1.10.2-pythons2n3.tar.gz (timeout 10 seconds)
3 packages and 1 specfiles checked; 22 errors, 0 warnings.


Usually, the scripts in %{python_sitelib} and %{python3_sitelib} don't need to be executable and don't need a shebang. Please remove the shebangs, and your package is ready for a full review.

Comment 6 Mario Blättermann 2013-07-29 07:14:41 UTC
Any news here...?

Comment 7 Ville Skyttä 2013-09-09 14:51:32 UTC
Looks like another review overtook this one, see bug 998738

Comment 8 Christopher Meng 2013-09-09 14:57:50 UTC

*** This bug has been marked as a duplicate of bug 998738 ***

Comment 9 Pete Travis 2013-09-09 15:03:20 UTC
Sorry I missed this, José - I would welcome you as a comaintainer.


Note You need to log in before you can comment on or make changes to this bug.