Red Hat Bugzilla – Bug 239526
Review Request: python-gammu - Python bindings for Gammu
Last modified: 2007-11-30 17:12:04 EST
Spec URL: http://download.tuxfamily.org/lxtnow/fedora/SPECS/python-gammu.spec
SRPM URL: http://download.tuxfamily.org/lxtnow/temp/python-gammu-0.19-1.src.rpm
Python bindings for Gammu library.
It currently does not support all Gammu features,but range of covered functions is increasing,if you need some specific, feel free to use bug tracking
system for feature requests.
So this package is also having similar rpmlint output like gammu have.
Got spurious-executable-perm and doc-file-depencecy with rpmlint 0.80 version.
Any comments on why both warnings should be ignored?
those are exemples script which could be check for extras use (such as manage
upstream isn't more explicit about them... :(
but here rpmlint is showing doc-file-dependency warning...
Is gammu-python really using these scripts or its for end-user to explicitly use
them? if that is then you can chmod 644 to those scripts.
It's typically for end-user and sure these can be set to 0644.
i actually fix that but i need to import first new gammu release before add the
updated release of python-gammu which require gammu-1.10.7 to be able to build.
Here is the updated release 0.20 with its fix.
you'll find out srpm, spec and build.log files from link below.
ping : Parag ?
will do review by tomorrow
rpmlint on SRPM reports
W: python-gammu mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line 4)
The specfile mixes use of spaces and tabs for indentation, which is a
cosmetic annoyance. Use either spaces or tabs for indentation, not both.
you can correct this at time of cvs import.
+ package builds in mock (development i386).
- rpmlint is NOT silent for SRPM
+ rpmlint is silent for RPM.
+ source files match upstream url
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no static libraries.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Provides: Core.so
+ Requires: gammu libGammu.so.1 libc.so.6 libc.so.6(GLIBC_2.0)
libc.so.6(GLIBC_2.1) libc.so.6(GLIBC_2.1.3) libc.so.6(GLIBC_2.3)
libc.so.6(GLIBC_2.3.4) libc.so.6(GLIBC_2.4) libm.so.6 libpthread.so.0
libpython2.5.so.1.0 python(abi) = 2.5 rtld(GNU_HASH)
Don't forget to solve rpmlint warning on SRPM.
I will fix that before request CVSsync, thanks Parag
New Package CVS Request
Package Name: python-gammu
Short Description: Python bindings for Gammu
Branches: FC-5 FC-6 F-7 EL-4 EL-5 Devel
imported and built.
Thanks to all