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 bugzilla.redhat.com.
Bug 175623 - Review Request: yaz - Z39.50/SRW/SRU programs
Summary: Review Request: yaz - Z39.50/SRW/SRU programs
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: David Lawrence
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-12-13 14:47 UTC by Konstantin Ryabitsev
Modified: 2014-03-25 11:56 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-07 21:50:18 UTC
Type: ---
Embargoed:
i: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Konstantin Ryabitsev 2005-12-13 14:47:15 UTC
Spec Name or Url: http://linux.duke.edu/~icon/misc/fe/yaz.spec
SRPM Name or Url: http://linux.duke.edu/~icon/misc/fe/yaz-2.1.10-1.src.rpm
Description:
YAZ is a programmersâ toolkit supporting the development of Z39.50/SRW/SRU clients and servers. Z39.50-2003 (version 3) as well as SRW/SRU version 1.1 are supported in both the client and server roles.

Comment 1 John Mahowald 2006-02-25 23:00:02 UTC
Build failed, devel x86_64

checking for working tcpd.h... no
configure: error: tcpd development libraries missing


Comment 2 Jason Tibbitts 2006-06-14 14:19:29 UTC
Adding back my comment that was lost in the crash:

------- Additional Comments From tibbs.edu  2006-06-10 16:44 EST -------
icon, are you still interested in packaging this?  I have to say, the
description is terribly non-descriptive.  Even the upstream website doesn't
fother to say what Z39.50 is.  Perhaps adding the following (cribbed from
wikipedia) would help a bit:

Z39.50 is a client server protocol for searching and retrieving information from
remote computer databases.

This does, however, properly build in mock (x86_64, development).  If icon is
still interested and updates to the current version (2.1.20), I could go ahead
with a review of this.

BTW, this package has rpath problems on x86_64.  This seems to be fixed by the
usual technique of addingg BR: libtool and then adding LIBTOOL=/usr/bin/libtool
on the make line.  I have no idea if this breaks anything.  There seems to be an
included test suite but no %check section in the spec.


Comment 3 Konstantin Ryabitsev 2006-06-14 15:31:08 UTC
Yep, I'm still alive. I'll take a look at it briefly.

Comment 4 Konstantin Ryabitsev 2006-06-20 19:31:38 UTC
http://linux.duke.edu/~icon/misc/fe/yaz.spec
http://blues.mcgill.ca/~icon/fe/yaz-2.1.22-1.src.rpm

* Tue Jun 20 2006 Konstantin Ryabitsev <icon> - 2.1.22-1
- Version 2.1.22
- Libtoolize correctly
- BuildRequire libxslt
- BuildRequire tcp_wrappers
- Enable pth in configure
- Add %%check routine

Comment 5 Jason Tibbitts 2006-06-21 04:02:51 UTC
Builds fine in mock (x86_64, development) and rpmlint is quiet.

* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.  License text included in package.
* source files match upstream:
   2d401ea471a87e7a056ea2df9e2d9d14  yaz-2.1.22.tar.gz
* latest version is being packaged.
* BuildRequires are proper.
* package builds in mock (x86_64, development).
* rpmlint is silent.
* final provides and requires are sane:

  libyaz-2.1.22-1.fc6.x86_64.rpm
   libyaz.so.2()(64bit)
   libyazthread.so.2()(64bit)
   libyaz = 2.1.22-1.fc6
  =
   /sbin/ldconfig
   libcrypto.so.6()(64bit)
   libexslt.so.0()(64bit)
   libgcrypt.so.11()(64bit)
   libgpg-error.so.0()(64bit)
   libssl.so.6()(64bit)
   libwrap.so.0()(64bit)
   libxml2.so.2()(64bit)
   libxslt.so.1()(64bit)
   libyaz.so.2()(64bit)
   libyazthread.so.2()(64bit)
   libz.so.1()(64bit)

  libyaz-devel-2.1.22-1.fc6.x86_64.rpm
   libyaz-devel = 2.1.22-1.fc6
  =
   /bin/sh
   libxml2-devel
   libyaz = 2.1.22-1.fc6
   libyaz.so.2()(64bit)
   libyazthread.so.2()(64bit)
   openssl-devel
   readline-devel
   
  yaz-2.1.22-1.fc6.x86_64.rpm
   yaz = 2.1.22-1.fc6
  =
   libcrypto.so.6()(64bit)
   libexslt.so.0()(64bit)
   libgcrypt.so.11()(64bit)
   libgpg-error.so.0()(64bit)
   libhistory.so.5()(64bit)
   libncurses.so.5()(64bit)
   libreadline.so.5()(64bit)
   libssl.so.6()(64bit)
   libwrap.so.0()(64bit)
   libxml2.so.2()(64bit)
   libxslt.so.1()(64bit)
   libyaz.so.2()(64bit)
   libz.so.1()(64bit)

* shared libraries are present; ldconfig is called and unversioned .so files are
in the -devel package.
* package is not relocatable.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* %clean is present.
* %check is present and all tests pass:
   ===================
   All 21 tests passed
   ===================
* scriptlets present and OK (ldconfig calls)
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* headers present in -devel package.
* no pkgconfig files.
* no libtool .la droppings.
* not a GUI app.

APPROVED

Comment 6 Jason Tibbitts 2006-07-07 16:44:13 UTC
This package has been in the repo for a couple of weeks now; any reason this bug
hasn't been closed?

Comment 7 Konstantin Ryabitsev 2006-07-07 21:50:18 UTC
Just lack of attention. :) Closing, thanks.

Comment 8 Konstantin Ryabitsev 2007-04-01 22:06:21 UTC
Please create EPEL branches:

Branches: EL-4, EL-5

Comment 9 Jens Petersen 2007-04-02 12:31:27 UTC
done

Comment 10 Christopher Meng 2014-03-25 09:23:49 UTC
Package Change Request
======================
Package Name: yaz
New Branches: epel7
Owners: cicku

Comment 11 Gwyn Ciesla 2014-03-25 11:56:18 UTC
Git done (by process-git-requests).


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