Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at
Bug 1728373 (python-glymur) - Review Request: python-glymur - Glymur is a Python interface for JPEG 2000
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, NeuroFedora
TreeView+ depends on / blocked
Reported: 2019-07-09 19:36 UTC by Luis Segundo
Modified: 2023-09-14 05:31 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: ---

Attachments (Terms of Use)

Description Luis Segundo 2019-07-09 19:36:42 UTC
Spec URL:

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

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/
+%{__python3} -m unittest discover

I had to create 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

Comment 7 Red Hat Bugzilla 2023-09-14 05:31:41 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 1000 days

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