[Bug 699902] Review Request: python-manuel - Build tested documentation

[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=699902

--- Comment #3 from Robin Lee <robinlee.sysu at gmail.com> 2011-04-27 13:08:21 EDT ---
(In reply to comment #2)
> Thanks for the review!
> 
> (In reply to comment #1)
> > [-] According to setup.py, this package does not require python-zope-testing at
> > runtime.
> 
> I did a grep for imports and found this:
> 
> import zope.testing.renormalizing
> 
> in tests.py, hence the Requires.  On a second look, though, that file is only
> used to test manuel itself.  It probably shouldn't be installed, then.  I'll
> ask upstream about that.
Zope packages usually use [main_namespace].tests for the test suite. That means
[main_namespace].tests and any sub-namespace of it will be reached only when
test runs. I hope you can confirm this point with upstream.

For package that comes with vast test suite, those module files will even be
excluded from binary rpm as suggested by some packagers. python-ZODB3 is an
example.

-- 
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.