Bug 1005599 - Review Request: erfa - Essential Routines for Fundamental Astronomy
Summary: Review Request: erfa - Essential Routines for Fundamental Astronomy
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Christopher Meng
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-09-08 22:22 UTC by Sergio Pascual
Modified: 2013-09-17 21:22 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-17 21:22:55 UTC
Type: ---
i: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Sergio Pascual 2013-09-08 22:22:26 UTC
Spec URL: http://guaix.fis.ucm.es/~spr/erfa.spec
SRPM URL: http://guaix.fis.ucm.es/~spr/erfa-0.0.1-1.fc19.src.rpm
Description: ERFA is a C library containing key algorithms for astronomy. It is 
based on the SOFA library published by the International Astronomical Union (IAU).
Fedora Account System Username: sergiopr

Comment 1 Christopher Meng 2013-09-08 23:02:25 UTC
- devel missing %{?_isa} tag.

- Why should we need pkgconfig when installing devel?

- devel %summary is not perfect, current one:

Headers for developing programs that will use %{name}

Ah there are not only headers...

Suggestion:

%package        devel
Summary:        Development files for %{name}
Requires:       %{name}%{?_isa} = %{version}-%{release}

%description    devel
The %{name}-devel package contains libraries and header files for
developing applications that use %{name}.

- You don't need to add smpflag during installation.

Comment 2 Sergio Pascual 2013-09-09 11:17:01 UTC
Updated.

For the pkgconfig thing, -devel contains erfa.pc that it's installed in %{_libdir}/pkgconfig. This directory belongs to pkgconfig. If I leave out the dependecy, %{_libdir}/pkgconfig would be unowned.

Spec URL: http://guaix.fis.ucm.es/~spr/erfa.spec
SRPM URL: http://guaix.fis.ucm.es/~spr/erfa-0.0.1-2.fc19.src.rpm

Comment 3 Michael Schwendt 2013-09-09 12:08:30 UTC
A dependency on /usr/bin/pkg-config is added automatically for quite some time.

Comment 4 Christopher Meng 2013-09-09 12:52:47 UTC
(In reply to Michael Schwendt from comment #3)
> A dependency on /usr/bin/pkg-config is added automatically for quite some
> time.

Yes, that's what I want to say, thanks Michael.

And you still don't change the description of -devel

Comment 5 Sergio Pascual 2013-09-09 13:44:09 UTC
(In reply to Christopher Meng from comment #4)
> (In reply to Michael Schwendt from comment #3)
> > A dependency on /usr/bin/pkg-config is added automatically for quite some
> > time.
> 
> Yes, that's what I want to say, thanks Michael.
> 

I see, I have removed the Requires: pkgconfig

> And you still don't change the description of -devel

I have updated the Summary field also.

Spec URL: http://guaix.fis.ucm.es/~spr/erfa.spec
SRPM URL: http://guaix.fis.ucm.es/~spr/erfa-0.0.1-3.fc19.src.rpm

Comment 6 Christopher Meng 2013-09-11 23:43:01 UTC
APPROVED.

No issues found.

Comment 7 Sergio Pascual 2013-09-12 10:53:36 UTC
New Package SCM Request
=======================
Package Name: erfa
Short Description: Essential Routines for Fundamental Astronomy
Owners: sergiopr
Branches: f18 f19 f20 el5 el6
InitialCC:

Comment 8 Gwyn Ciesla 2013-09-12 12:20:40 UTC
Git done (by process-git-requests).


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