Bug 1312407 - Review Request: python-pybtex - BibTeX-compatible bibliography processor written in Python
Summary: Review Request: python-pybtex - BibTeX-compatible bibliography processor writ...
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:
Blocks: 1312409
TreeView+ depends on / blocked
 
Reported: 2016-02-26 16:05 UTC by Jerry James
Modified: 2016-03-05 15:59 UTC (History)
2 users (show)

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


Attachments (Terms of Use)

Description Jerry James 2016-02-26 16:05:11 UTC
Spec URL: https://jjames.fedorapeople.org/python-pybtex/python-pybtex.spec
SRPM URL: https://jjames.fedorapeople.org/python-pybtex/python-pybtex-0.19-1.fc24.src.rpm
Fedora Account System Username: jjames
Description: Pybtex is a BibTeX-compatible bibliography processor written in Python. Pybtex aims to be 100% compatible with BibTeX. It accepts the same command line options, fully supports BibTeX’s .bst styles and produces byte-identical output.

Additionally:
- Pybtex is Unicode-aware.
- Pybtex supports bibliography formats other than BibTeX.
- It is possible to write formatting styles in Python.
- As a bonus, Pythonic styles can produce HTML, Markdown and other markup besides the usual LaTeX.
Pybtex also includes a Python API for managing bibliographies from Python.

Comment 1 Zbigniew Jędrzejewski-Szmek 2016-02-28 02:26:05 UTC
"Pybtex is Unicode-aware." — thank you. I'd love not to have to fight with bibtex anymore.

The same as with the other package: I don't think you need to preserve timestamps of files that have been modified.

Would it be possible to build both versions from the same build directory?

Would it be possible to replace the bundled jquery with the system one? Same for js-underscore?

You can use the following trick to simplify the %files section:

%files -n python2-{%srcname}
...

%if 0{?with_py3}
...
%endif

%{_bindir}/pybtex*
%{_mandir}/man1/pybtex*

This way the last two lines will fall into python2- subpackage if there's no python3- subpackage.

Comment 2 Zbigniew Jędrzejewski-Szmek 2016-02-28 02:56:49 UTC
(In reply to Zbigniew Jędrzejewski-Szmek from comment #1)
> Would it be possible to build both versions from the same build directory?
OK, I see that 2to3 is used, so the answer is most likely no.

> Would it be possible to replace the bundled jquery with the system one? Same
> for js-underscore?
I see that it is used in documentation only. I withdraw this question.

Comment 3 Jerry James 2016-03-03 03:49:26 UTC
(In reply to Zbigniew Jędrzejewski-Szmek from comment #1)
> "Pybtex is Unicode-aware." — thank you. I'd love not to have to fight with
> bibtex anymore.

:-)

> The same as with the other package: I don't think you need to preserve
> timestamps of files that have been modified.

Okay, done.

> You can use the following trick to simplify the %files section:

Nice trick, thanks.  I have applied it.  I also fixed the nosetests invocations.  New URLs:

Spec URL: https://jjames.fedorapeople.org/python-pybtex/python-pybtex.spec
SRPM URL: https://jjames.fedorapeople.org/python-pybtex/python-pybtex-0.19-2.fc25.src.rpm

Comment 4 Zbigniew Jędrzejewski-Szmek 2016-03-03 13:38:44 UTC
fedora-review says:
- Large documentation must go in a -doc subpackage. Large could be size
  (~1MB) or number of files.
  Note: Documentation size is 1638400 bytes in 94 files.
  See:
  http://fedoraproject.org/wiki/Packaging/Guidelines#PackageDocumentation
and I have to agree. No need to have the same stuff twice, so a -doc subpackage shared between both python versions would be nice.

+ license file is present, %license is used
+ license is acceptable (MIT)
+ latest version
+ %check is present, checks pass
+ provides/requires look OK
+ no scriptlets present or necessary
+ rpmlint: only false positive spelling errors
+ executables prefer python3

Package is APPROVED. Please split out -doc.

Comment 5 Jerry James 2016-03-04 21:17:23 UTC
Okay, I will split out -doc.  Thank you!  New package requested.

Comment 6 Gwyn Ciesla 2016-03-04 22:25:31 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/python-pybtex


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