Bug 1312410 - Review Request: python-sphinxcontrib-bibtex - Sphinx extension for BibTeX style citations
Summary: Review Request: python-sphinxcontrib-bibtex - Sphinx extension for BibTeX sty...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1304217 1312406 1312408 1312409
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-02-26 16:05 UTC by Jerry James
Modified: 2016-03-05 16:00 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-03-05 16:00:42 UTC
Type: ---
Embargoed:
zbyszek: fedora-review+


Attachments (Terms of Use)

Description Jerry James 2016-02-26 16:05:28 UTC
Spec URL: https://jjames.fedorapeople.org/python-sphinxcontrib-bibtex/python-sphinxcontrib-bibtex.spec
SRPM URL: https://jjames.fedorapeople.org/python-sphinxcontrib-bibtex/python-sphinxcontrib-bibtex-0.3.3-1.fc24.src.rpm
Fedora Account System Username: jjames
Description: Sphinxcontrib-bibtex is a Sphinx extension for BibTeX style citations.

This extension allows BibTeX citations to be inserted into documentation generated by Sphinx, via a bibliography directive, and a cite role, which work similarly to LaTeX's thebibliography environment and \cite command.

For formatting, the extension relies on pybtex, written by Andrey Golovizin.

Comment 1 Zbigniew Jędrzejewski-Szmek 2016-02-28 02:58:11 UTC
I'll review this when deps are done.

Comment 2 Jerry James 2016-03-03 04:53:31 UTC
Thank you so much for picking up all of these reviews.  I really appreciate it.  Since you haven't started on this one yet, I have sneakily made the same changes you requested in the other python packages.  New URLs:

Spec URL: https://jjames.fedorapeople.org/python-sphinxcontrib-bibtex/python-sphinxcontrib-bibtex.spec
SRPM URL: https://jjames.fedorapeople.org/python-sphinxcontrib-bibtex/python-sphinxcontrib-bibtex-0.3.3-1.fc25.src.rpm

Comment 3 Zbigniew Jędrzejewski-Szmek 2016-03-03 15:36:31 UTC
nosetests -v, so the test names are printed.

/usr/share/doc/python{2,3}-sphinxcontrib-bibtex/ are 1MB each, it would be nice to have a shared -doc subpackage.

+ latest version
+ license is acceptable (BSD)
+ license file is present, %license is used
+ provides/requires look OK
+ %python_provide is used
+ %check is present and passes
+ no scriptlets required or present
+ rpmlint is OK:
python2-sphinxcontrib-bibtex.noarch: W: spelling-error %description -l en_US thebibliography -> the bibliography, the-bibliography, bibliography
python2-sphinxcontrib-bibtex.noarch: W: spelling-error %description -l en_US pybtex -> Playtex
python3-sphinxcontrib-bibtex.noarch: W: spelling-error %description -l en_US thebibliography -> the bibliography, the-bibliography, bibliography
python3-sphinxcontrib-bibtex.noarch: W: spelling-error %description -l en_US pybtex -> Playtex
python-sphinxcontrib-bibtex.src: W: spelling-error %description -l en_US thebibliography -> the bibliography, the-bibliography, bibliography
python-sphinxcontrib-bibtex.src: W: spelling-error %description -l en_US pybtex -> Playtex
python-sphinxcontrib-bibtex.src:67: W: unversioned-explicit-provides bundled(jquery)
python-sphinxcontrib-bibtex.src:68: W: unversioned-explicit-provides bundled(js-underscore)
python-sphinxcontrib-bibtex.src:83: W: unversioned-explicit-provides bundled(jquery)
python-sphinxcontrib-bibtex.src:84: W: unversioned-explicit-provides bundled(js-underscore)
3 packages and 0 specfiles checked; 0 errors, 10 warnings.
+ package works fine, nice documentation

Package is APPROVED. Please consider splitting out -doc subpackage.

Comment 4 Jerry James 2016-03-04 21:27:16 UTC
Okay, I will add -v to the nosetests invocations and split out -doc.  Thank you very much for reviewing this pile of packages!  I appreciate your effort.

Comment 5 Gwyn Ciesla 2016-03-04 22:26:18 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/python-sphinxcontrib-bibtex


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