[Bug 523034] Review Request: python-setuptools_trial - Setuptools plugin that makes unit tests execute with trial instead of pyunit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=523034

Toshio Ernie Kuratomi <a.badger@xxxxxxxxx> changed:

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

--- Comment #8 from Toshio Ernie Kuratomi <a.badger@xxxxxxxxx> 2010-11-04 10:14:36 EDT ---
NEEDSWORK

Good:
* Named according to the naming guidelines (there's an underscore in all the
  upstream references to this name.)
* license is BSD which is approved
* spec file is readable
* Source matches upstream
* No locale files
* No shared libraries
* Not relocatable
* No file listed multiple times
* Permissions set properly
* Package consistently uses macros
* Code not content
* Nothing in doc affects runtime
* Not a GUI application
* Does not bundle libraries in the built rpm but see below
* filenames are valid utf8
* Does not own files owned by other packages
* Owns all directories it creates

Needswork:
* Does not build in koji.  The build looks for the setuptools_darcs package.
  This does not appear to be needed (only for creating sdists, not for the
  built package that we're doing so you can remove the line from the setup.py
that adds this to requires.
* setuptools_darcs code is present in the tarball.  However, it is not being
  used in the build.  The FPC is looking at new guidelines that say to remove
  such code (to make sure that it isn't being used in the build) so be ready
  for that potential guideline change.
  https://fedoraproject.org/wiki/PackagingDrafts/Treatment_Of_Bundled_Libraries

Rpmlint:
srpm and binary rpm share these warnings:

python-setuptools_trial.src: W: spelling-error %description -l en_US Setuptools
-> Setup tools, Setup-tools, Setups
python-setuptools_trial.src: W: spelling-error %description -l en_US plugin ->
plug in, plug-in, plugging
python-setuptools_trial.src: W: spelling-error %description -l en_US pyunit ->
pyuria, punitive, impunity

These are all common words in computer jargon or proper names so these are
false positives.


Fix the building problems and I'll give it one more look.

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]