Bug 1384242 - Review Request: python-astropy-helpers - Utilities for building and installing Astropy and affiliated package
Summary: Review Request: python-astropy-helpers - Utilities for building and installin...
Keywords:
Status: CLOSED NOTABUG
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: 1331062 1356123
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-10-12 22:35 UTC by Orion Poplawski
Modified: 2016-12-08 09:50 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-11-12 18:51:25 UTC
Type: Bug


Attachments (Terms of Use)

Description Orion Poplawski 2016-10-12 22:35:58 UTC
Spec URL: https://www.cora.nwra.com/~orion/fedora/python-astropy-helpers.spec
SRPM URL: https://www.cora.nwra.com/~orion/fedora/python-astropy-helpers-1.2-1.fc26.src.rpm
Description:
This project provides a Python package, astropy_helpers, which includes many
build, installation, and documentation-related tools used by the Astropy
project, but packaged separately for use by other projects that wish to
leverage this work.

Fedora Account System Username: orion

scratch build - http://koji.fedoraproject.org/koji/taskinfo?taskID=16067724

Comment 1 Dominika Krejčí 2016-10-14 08:47:21 UTC
Hi Orion, links to SPEC and SRPM are invalid.

Comment 2 Orion Poplawski 2016-10-14 13:10:48 UTC
Sorry, should be working now.

Comment 3 Dominika Krejčí 2016-10-21 11:25:17 UTC
Is there any special reason why do you use `%{pythonX_pkgversion}` macro? I think you could use just 3 or 2, e.g. `python3-devel`.

Could you also specify Python 2 Requires?
python-devel -> python2-devel
python-setuptools -> python2-setuptools
pytest -> python2-pytest
numpy -> python2-numpy
python-shipnx -> python2-sphinx

Could you be please more specific in the `%files` section and do not use just `%{pythonX_sitelib}/*`, but for example this? %{python3_sitelib}/*astropy_helpers*.egg-info/
%{python3_sitelib}/astropy_helpers/

Comment 4 Orion Poplawski 2016-10-21 20:05:18 UTC
(In reply to Dominika Krejčí from comment #3)
> Is there any special reason why do you use `%{pythonX_pkgversion}` macro? I
> think you could use just 3 or 2, e.g. `python3-devel`.

For EPEL support - see https://fedoraproject.org/wiki/PackagingDrafts:Python3EPEL

> Could you also specify Python 2 Requires?
> python-devel -> python2-devel
> python-setuptools -> python2-setuptools
> pytest -> python2-pytest
> numpy -> python2-numpy
> python-shipnx -> python2-sphinx

Same reason - EPEL uses the old names.

> Could you be please more specific in the `%files` section and do not use
> just `%{pythonX_sitelib}/*`, but for example this?
> %{python3_sitelib}/*astropy_helpers*.egg-info/
> %{python3_sitelib}/astropy_helpers/

How about:

%{python3_sitelib}/astropy_helpers*

Spec URL: https://www.cora.nwra.com/~orion/fedora/python-astropy-helpers.spec
SRPM URL: https://www.cora.nwra.com/~orion/fedora/python-astropy-helpers-1.2-2.el7.src.rpm

* Fri Oct 21 2016 Orion Poplawski <orion.com> - 1.2-2
- Be more explicit in %%files

Comment 5 Zbigniew Jędrzejewski-Szmek 2016-11-12 18:51:25 UTC
Oops, package already exists: https://admin.fedoraproject.org/pkgdb/package/rpms/python-astropy-helpers/.


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