Bug 1003960 - Review Request: glite-lb-client - gLite Logging and Bookkeeping client library
Summary: Review Request: glite-lb-client - gLite Logging and Bookkeeping client library
Keywords:
Status: CLOSED ERRATA
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: 1001104
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-09-03 14:55 UTC by František Dvořák
Modified: 2013-12-24 17:57 UTC (History)
2 users (show)

Fixed In Version: glite-lb-client-6.0.8-2.fc20
Clone Of:
Environment:
Last Closed: 2013-12-19 07:04:46 UTC
Type: ---
Embargoed:
i: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description František Dvořák 2013-09-03 14:55:37 UTC
Spec URL: http://scientific.zcu.cz/fedora/glite-lb-client-6.0.7-1/glite-lb-client.spec
SRPM URL: <http://scientific.zcu.cz/fedora/glite-lb-client-6.0.7-1/glite-lb-client-6.0.7-1.fc21.src.rpm
Description: glite-lb-client is the library implementing client side of gLite Logging and Bookkeeping API. glite-lb-client-progs package contains client programs and examples for gLite L&B.
Fedora Account System Username: valtri

Notes:
- I'm upstream maintainer
- there are already three patches (based on issues from other reviews), they will be included in the next release
- koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5886873

Comment 2 Christopher Meng 2013-12-05 11:44:03 UTC
Let me help glite packages. ;)

Comment 3 Christopher Meng 2013-12-06 17:23:58 UTC
1. Requires:       glite-jobid-api-cpp-devel

Where is the %{?_isa} at the ending?

2. install -m 0644 LICENSE project/ChangeLog $RPM_BUILD_ROOT%{_pkgdocdir}

Better use %doc in %files

3. %{_mandir}/man1/glite-lb-notify.1.gz
%{_mandir}/man1/glite-lb-logevent.1.gz

Please use glob:

%{_mandir}/man1/glite-lb-notify.1*
%{_mandir}/man1/glite-lb-logevent.1*

So RPM will help compress it if it doesn't.

---------------------
PACKAGE APPROVED.

Fix issues above before SCM import.

Comment 4 František Dvořák 2013-12-06 22:09:10 UTC
Thanks for the review! This is milestone, gLite L&B clients in native Fedora/EPEL repositories. :-)

Fixing/explaining issues:

1. glite-jobid-api-cpp-devel is noarch package (contains only header files), so %{?_isa} has not been used

2. fixed (also in other gLite packages)

3. there is problem when combining %doc with listing installed documentation files; it is working OK on Fedora, but rpmbuild fails for EPEL 5/6:

error: File not found: /builddir/build/BUILDROOT/glite-lb-client-6.0.8-1.git20131206.el6.x86_64/usr/share/doc/glite-lb-client-6.0.8/examples
error: File not found by glob: /builddir/build/BUILDROOT/glite-lb-client-6.0.8-1.git20131206.el6.x86_64/usr/share/doc/glite-lb-client-6.0.8/examples/*
    File not found: /builddir/build/BUILDROOT/glite-lb-client-6.0.8-1.git20131206.el6.x86_64/usr/share/doc/glite-lb-client-6.0.8/examples
    File not found by glob: /builddir/build/BUILDROOT/glite-lb-client-6.0.8-1.git20131206.el6.x86_64/usr/share/doc/glite-lb-client-6.0.8/examples/*

I added a comment to spec.


The new version:

Spec URL: http://scientific.zcu.cz/fedora/glite-lb-client-6.0.8-2/glite-lb-client.spec
SRPM URL: http://scientific.zcu.cz/fedora/glite-lb-client-6.0.8-2/glite-lb-client-6.0.8-2.fc21.src.rpm

Comment 5 Christopher Meng 2013-12-07 03:55:30 UTC
Fine. You can request SCM now.

Comment 6 František Dvořák 2013-12-07 04:53:36 UTC
New Package SCM Request
=======================
Package Name: glite-lb-client
Short Description: gLite Logging and Bookkeeping client library and programs
Owners: valtri
Branches: f18 f19 f20 el6
InitialCC:

Comment 7 Gwyn Ciesla 2013-12-09 13:20:36 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2013-12-09 18:12:36 UTC
glite-lb-client-6.0.8-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/glite-lb-client-6.0.8-2.fc20

Comment 9 Fedora Update System 2013-12-09 18:13:37 UTC
glite-lb-client-6.0.8-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/glite-lb-client-6.0.8-2.fc19

Comment 10 Fedora Update System 2013-12-09 18:14:30 UTC
glite-lb-client-6.0.8-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/glite-lb-client-6.0.8-2.fc18

Comment 11 Fedora Update System 2013-12-09 18:15:44 UTC
glite-lb-client-6.0.8-2.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/glite-lb-client-6.0.8-2.el6

Comment 12 Fedora Update System 2013-12-09 20:24:56 UTC
glite-lb-client-6.0.8-2.fc20 has been pushed to the Fedora 20 testing repository.

Comment 13 Fedora Update System 2013-12-19 07:04:46 UTC
glite-lb-client-6.0.8-2.fc20 has been pushed to the Fedora 20 stable repository.

Comment 14 Fedora Update System 2013-12-19 07:18:21 UTC
glite-lb-client-6.0.8-2.fc19 has been pushed to the Fedora 19 stable repository.

Comment 15 Fedora Update System 2013-12-19 07:18:41 UTC
glite-lb-client-6.0.8-2.fc18 has been pushed to the Fedora 18 stable repository.

Comment 16 Fedora Update System 2013-12-24 17:57:07 UTC
glite-lb-client-6.0.8-2.el6 has been pushed to the Fedora EPEL 6 stable repository.


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