Bug 1728373 (python-glymur) - Review Request: python-glymur - Glymur is a Python interface for JPEG 2000 [NEEDINFO]
Summary: Review Request: python-glymur - Glymur is a Python interface for JPEG 2000
Alias: python-glymur
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Tomas Korbar
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: FE-DEADREVIEW fedora-neuro
TreeView+ depends on / blocked
Reported: 2019-07-09 19:36 UTC by Luis Segundo
Modified: 2019-12-16 18:01 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2019-12-16 09:03:06 UTC
Type: ---
tkorbar: needinfo? (luis)

Attachments (Terms of Use)

Description Luis Segundo 2019-07-09 19:36:42 UTC
Spec URL: https://blackfile.fedorapeople.org/glymur/python-glymur.spec
SRPM URL: https://blackfile.fedorapeople.org/glymur/python-glymur-0.8.18-1.fc30.src.rpm

Description: Glymur is an interface to the OpenJPEG library which allows one to read and write JPEG 2000 files from Python. Glymur supports both reading and writing of JPEG 2000 images, but writing JPEG 2000 images is currently limited to images that can fit in memory. Glymur can read images using OpenJPEG library versions as far back as 1.3, but it is strongly recommended to use at least version 2.1.2.

Fedora Account System Username:blackfile

Comment 1 Tomas Korbar 2019-07-10 12:31:14 UTC
Hi Luis,
I will take this package review.
First issue with this package is that it fails to build in rawhide (see [0]).
[0] https://koji.fedoraproject.org/koji/taskinfo?taskID=36166641

Comment 2 Luis Segundo 2019-07-10 13:27:27 UTC
Thanks Tomás, I'm going to check

Comment 4 Tomas Korbar 2019-07-11 09:37:23 UTC
Great! Now the review.
 16 BuildRequires:  python3-devel
 17 BuildRequires:  python3dist(numpy) >= 1.7.1
 18 BuildRequires:  python3dist(setuptools)
 19 BuildRequires:  python3dist(sphinx)
 20 BuildRequires:  python3dist(numpydoc)
 21 BuildRequires:  python3-sphinx_rtd_theme
^^^ You should remove these build requirements for the unversioned python package. They are not needed.
Look at example on python packaging guidelines at [0].
 24 %global debug_package %{nil}
^^^ Please remove this too. It is obsolete for python package.
 29 %package -n     python3-%{pypi_name}
 30 Summary:        %{summary}
 31 %{?python_provide:%python_provide python3-%{pypi_name}}
 33 Requires:       python3dist(numpy) >= 1.7.1
 34 Requires:       python3dist(setuptools)
^^^ Please list all build and runtime requirements here.
The upstream has working tests so run them in %check phase.
I tried them so i can give you an advice.
+touch tests/data/__init__.py
+%{__python3} -m unittest discover

I had to create __init__.py to prevent FileNotFoundError.
Maybe you will find some better way but if you will not then contact please upstream and discuss a solution of this with them.
 64 %{python3_sitelib}/*-py?.?.egg-info
^^^ At last but not least please change this regex to something more accurate.
Something like this ^. Just to be sure that nothing unexpected will happen.

Comment 5 Tomas Korbar 2019-12-05 07:32:45 UTC
Hi Luis,
please respond or i will mark this review as stalled.

Comment 6 Luis Segundo 2019-12-16 18:01:14 UTC
Hi Tomás, in the next few days I will be updating the changes, thanks

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