[Bug 645604] Review Request: python-gflags - Commandline flags module for Python

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645604

Thomas Spura <tomspur at fedoraproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #3 from Thomas Spura <tomspur at fedoraproject.org> 2010-10-22 08:21:10 EDT ---
rpmlint:
$ rpmlint ./python-gflags-1.4-1.fc14.src.rpm
./noarch/python-gflags-1.4-1.fc14.noarch.rpm 
python-gflags.src: W: spelling-error Summary(en_US) Commandline -> Command
line, Command-line, Commanding
python-gflags.src: W: spelling-error %description -l en_US google -> Google,
goggle, googly
python-gflags.src: W: spelling-error %description -l en_US commandline ->
command line, command-line, commanding
python-gflags.src: W: spelling-error %description -l en_US getopt -> get opt,
get-opt, getup
python-gflags.src: W: invalid-url Source0:
http://python-gflags.googlecode.com/files/python-gflags-1.4.tar.gz HTTP Error
404: Not Found
python-gflags.noarch: W: spelling-error Summary(en_US) Commandline -> Command
line, Command-line, Commanding
python-gflags.noarch: W: spelling-error %description -l en_US google -> Google,
goggle, googly
python-gflags.noarch: W: spelling-error %description -l en_US commandline ->
command line, command-line, commanding
python-gflags.noarch: W: spelling-error %description -l en_US getopt -> get
opt, get-opt, getup
python-gflags.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/gflags.py 0644L /usr/bin/env
python-gflags.noarch: W: no-manual-page-for-binary gflags2man
2 packages and 0 specfiles checked; 1 errors, 10 warnings.


Please fix the non-excecutable-script error, see:
http://fedoraproject.org/wiki/Packaging_tricks#Remove_shebang_from_Python_libraries

The rest is ignorable (invalid-url is false positive).

- license ok
- no libs
- no *.la
- rest also ok

___________________________

Please fix the non-executable-script error before importing.

___________________________


APPROVED

___________________________

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.