Bug 439337 - Review Request: python-sphinx - Python documentation generator
Summary: Review Request: python-sphinx - Python documentation generator
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: José Matos
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-03-28 03:46 UTC by Michel Lind
Modified: 2014-06-13 15:18 UTC (History)
4 users (show)

Fixed In Version: 0.3-1.fc9
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-07-06 06:14:46 UTC
Type: ---
Embargoed:
jamatos: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)
Patch to include more documentation (html and rst). (939 bytes, patch)
2008-04-16 11:46 UTC, José Matos
no flags Details | Diff

Description Michel Lind 2008-03-28 03:46:02 UTC
Spec URL: http://salimma.fedorapeople.org/for_review/python/python-sphinx.spec
SRPM URL: http://salimma.fedorapeople.org/for_review/python/python-sphinx-0.1.61950-1.fc9.src.rpm
Description:

Sphinx is a tool that makes it easy to create intelligent and
beautiful documentation for Python projects (or other documents
consisting of multiple reStructuredText sources), written by Georg
Brandl. It was originally created to translate the new Python
documentation, but has now been cleaned up in the hope that it will be
useful to many other projects.

Sphinx uses reStructuredText as its markup language, and many of its
strengths come from the power and straightforwardness of
reStructuredText and its parsing and translating suite, the Docutils.

Although it is still under constant development, the following
features are already present, work fine and can be seen "in action" in
the Python docs:

    * Output formats: HTML (including Windows HTML Help) and LaTeX,
      for printable PDF versions

    * Extensive cross-references: semantic markup and automatic links
      for functions, classes, glossary terms and similar pieces of
      information
    * Hierarchical structure: easy definition of a document tree, with
      automatic links to siblings, parents and children
    * Automatic indices: general index as well as a module index
    * Code handling: automatic highlighting using the Pygments highlighter
    * Various extensions are available, e.g. for automatic testing of
      snippets and inclusion of appropriately formatted docstrings.

Comment 1 Jason Tibbitts 2008-04-05 04:08:07 UTC
When I attempt to build this in mock, I get:

+ /usr/bin/python setup.py build
---------------------------------------------------------------------------
This script requires setuptools version 0.6c5 to run (even to display
help).  I will attempt to download it for you (from
http://cheeseshop.python.org/packages/2.5/s/setuptools/), but
you may need to enable firewall access for this script first.
I will start the download in 15 seconds.
(Note: if this machine does not have network access, please obtain the file
   http://cheeseshop.python.org/packages/2.5/s/setuptools/setuptools-0.6c5-py2.5.egg
and place it in this directory before rerunning this script.)
---------------------------------------------------------------------------

which obviously isn't going to work as mock cannot access the network.

Comment 2 Michel Lind 2008-04-09 18:57:52 UTC
Ah, missed some builddeps there. Will fix tonight, thanks. Just to clarify,
you're not officially reviewing this package, right?

Comment 4 José Matos 2008-04-14 15:53:38 UTC
I can review this package if Jason does not want and/or is too busy. :-)

Comment 5 José Matos 2008-04-14 16:22:55 UTC
Just for reference here it is my review:

Review for release 1.fc9:
* RPM name is OK
* Source Sphinx-0.1.61950.tar.gz is the same as upstream
* Builds fine in mock
* rpmlint looks OK
* File list looks OK
* The license is OK (and included in the package)

(30 checks have been run)

Comment 6 Jason Tibbitts 2008-04-15 16:46:19 UTC
It's fair game if I've not assigned the review to myself, so please go ahead.

Comment 7 José Matos 2008-04-16 11:44:42 UTC
Thanks Jason.

Michel the package looks OK there is a single problem:
rpmlint of python-sphinx:
python-sphinx.noarch: W: devel-file-in-non-devel-package
/usr/lib/python2.5/site-packages/sphinx/jinja/_speedups.c

Also IMHO you could build the documentation in the %build stage, there is a
Makefile in doc where the different targets are explained. The doc directory
with the rst documents is interesting by itself so I suggest to package it.

I suggest something like the patch that I will place in the next entry.

Comment 8 José Matos 2008-04-16 11:46:53 UTC
Created attachment 302585 [details]
Patch to include more documentation (html and rst).

Patch to include more documentation (html and rst).

Comment 9 Michel Lind 2008-05-02 18:01:05 UTC
Thanks Jose. Will incorporate the patch -- I should probably split it into a
-doc subpackage as well.


Comment 10 Michel Lind 2008-05-02 18:20:46 UTC
Problem fixed, documentation included and split into -doc. Thanks for the
suggestion -- ready when you are, Jose.

Spec URL: http://salimma.fedorapeople.org/for_review/python/python-sphinx.spec
SRPM URL:
http://salimma.fedorapeople.org/for_review/python/python-sphinx-0.1.61950-3.fc9.src.rpm



Comment 11 José Matos 2008-05-12 15:05:35 UTC
I like what I see. :-)

Just one other request, the new 0.3 version has been released, could you,
please, update the spec file?

As soon as you do that I will review and approve the package.

Comment 13 José Matos 2008-05-30 10:04:30 UTC
Two brief notes:

1) You define %pyver but you never use it.
2) The srpm spec file still has the exclusion part while the spec you present in
the entry above  does not have it. Of course the spec without the exclusion is
the right one.

These points are minor and can be fixed later when importing.

APPROVED.

Comment 14 Michel Lind 2008-06-05 14:01:44 UTC
Thanks, %{pyver} must have been a leftover from the default spec. Will
double-check when committing

New Package CVS Request
=======================
Package Name: python-sphinx
Short Description: Python documentation generator
Owners: salimma
Branches: EL-5 F-8 F-9
InitialCC:
Cvsextras Commits: yes




Comment 15 Kevin Fenzi 2008-06-05 16:17:48 UTC
cvs done.

Comment 16 Michel Lind 2008-06-06 16:38:57 UTC
Thanks.

Comment 17 Fedora Update System 2008-06-06 17:40:24 UTC
python-sphinx-0.3-1.fc9 has been submitted as an update for Fedora 9

Comment 18 Fedora Update System 2008-06-06 17:41:33 UTC
python-sphinx-0.3-1.fc8 has been submitted as an update for Fedora 8

Comment 19 Fedora Update System 2008-07-06 06:14:44 UTC
python-sphinx-0.3-1.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 20 Fedora Update System 2008-07-06 06:16:56 UTC
python-sphinx-0.3-1.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 21 Michel Lind 2011-12-08 15:27:27 UTC
Dan Radez is requesting this package for EPEL 6; I've tested that python-sphinx 1.0.x works fine there.

Package Change Request
======================
Package Name: python-sphinx
New Branches: el6
Owners: salimma radez
InitialCC:

Comment 22 Gwyn Ciesla 2011-12-08 15:29:40 UTC
Git done (by process-git-requests).

Comment 23 Michel Lind 2011-12-08 16:17:00 UTC
Thanks!


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