Bug 806926

Summary:	Review Request: tinycdb - Utility and library for manipulating constant databases
Product:	[Fedora] Fedora	Reporter:	Tom "spot" Callaway <tcallawa>
Component:	Package Review	Assignee:	Rex Dieter <rdieter>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	notting, package-review, rdieter
Target Milestone:	---	Flags:	rdieter: fedora-review+ gwync: fedora-cvs+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	tinycdb-0.77-4.fc17	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-04-12 02:59:23 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Tom "spot" Callaway 2012-03-26 14:08:19 UTC

Spec URL: http://spot.fedorapeople.org/tinycdb.spec
SRPM URL: http://spot.fedorapeople.org/tinycdb-0.77-4.fc17.src.rpm
Description: 
tinycdb is a small, fast and reliable utility and subroutine library for
creating and reading constant databases. The database structure is tuned
for fast reading.
Koji Rawhide Scratch Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3933211

Please note: This package was previously retired in Fedora due to FTBFS.

Comment 1 Rex Dieter 2012-03-26 15:02:04 UTC

i'll give it a whirl

Comment 2 Rex Dieter 2012-03-26 15:22:00 UTC

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

$ rpmlint *.src.rpm tinycdb tinycdb-devel
tinycdb.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libcdb.so.1 linux-vdso.so.1
2 packages and 0 specfiles checked; 0 errors, 1 warnings.
(mostly harmless)

SHOULD: please add comment near sed in %prep to explain it's purpose (it's only ~%95 obvious at first glance).

Naming: ok

Licensing: ok

Sources: ok
$ md5sum *.gz
c00e5fb96c30356ac3b67b2ab5d5641b  tinycdb_0.77.tar.gz

macros: ok

scriplets: ok

-devel deps: ok

The rest is fairly simple, and clean.


APPROVED.

Comment 3 Tom "spot" Callaway 2012-03-26 15:37:50 UTC

Package Change Request
======================
Package Name: tiny-cdb
New Branches: F16 F17
Owners: spot
InitialCC:

Comment 4 Gwyn Ciesla 2012-03-26 16:06:10 UTC

Should be a New Package Request, not Change.

Comment 5 Rex Dieter 2012-03-26 16:12:22 UTC

should be Package Name: tinycdb
I guess, which does match an existing one (adjusting Summary to match)

Comment 6 Tom "spot" Callaway 2012-03-26 17:27:44 UTC

Package Change Request
======================
Package Name: tinycdb
New Branches: F16 F17
Owners: spot
InitialCC:

Comment 7 Gwyn Ciesla 2012-03-26 18:01:26 UTC

Git done (by process-git-requests).

Unretired devel, take ownership of devel and f15.

Comment 8 Fedora Update System 2012-03-27 13:35:28 UTC

tinycdb-0.77-4.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/tinycdb-0.77-4.fc17

Comment 9 Fedora Update System 2012-03-28 06:00:33 UTC

tinycdb-0.77-4.fc17 has been pushed to the Fedora 17 testing repository.

Comment 10 Fedora Update System 2012-04-12 02:59:23 UTC

tinycdb-0.77-4.fc17 has been pushed to the Fedora 17 stable repository.