Bug 1098330 - Review Request: judy-fk - General purpose dynamic array with fixed-length keys
Summary: Review Request: judy-fk - General purpose dynamic array with fixed-length keys
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Lubomir Rintel
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-05-15 20:22 UTC by Miroslav Spousta
Modified: 2020-04-23 11:40 UTC (History)
3 users (show)

Fixed In Version: judy-fk-1.0.6-2.fc21
Clone Of:
Environment:
Last Closed: 2020-04-23 11:40:19 UTC
Type: ---
Embargoed:
lkundrak: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Miroslav Spousta 2014-05-15 20:22:06 UTC
Spec URL: http://www.ucw.cz/~qiq/rpms/judy-fk.spec
SRPM URL: http://www.ucw.cz/~qiq/rpms/judy-fk-1.0.6-1.fc20.src.rpm
Description: Judy is a C library that provides a state-of-the-art core technology that implements a sparse dynamic array. This package adds a modified SL library that accepts fixed length byte arrays.
Fedora Account System Username: qiq

Comment 1 Miroslav Spousta 2014-05-15 20:23:42 UTC
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6851820

Comment 2 Lubomir Rintel 2014-05-16 11:09:55 UTC
Taking this for a review.
Removing FE-NEEDSPONSOR, I'm going to sponsor Mirek.

Comment 3 Lubomir Rintel 2014-05-16 11:15:17 UTC
* Package named correctly
* Packaging the latest version
* License tag correct
* License is fine for fedora
* Builds fine in mock
* Rpmlint is happy
* SPEC file is clean and legible
* Filelist sane
* Requires sane
* Provides sane

APPROVED

One minor suggestion; these are no longer required in a modern RPM:

0.) Cleaning buildroot before install

rm -rf $RPM_BUILD_ROOT

1.) %clean section altogether

%clean
rm -rf $RPM_BUILD_ROOT

2.) %defattr

%defattr(-,root,root,-)

3.) BuildRoot tag

BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

Comment 4 Miroslav Spousta 2014-05-16 14:11:23 UTC
Spec URL: http://www.ucw.cz/~qiq/rpms/judy-fk.spec
SRPM URL: http://www.ucw.cz/~qiq/rpms/judy-fk-1.0.6-2.fc20.src.rpm

* Fri May 16 2014 Miroslav Spousta <miroslav.spousta> 1.0.6-2
- removal of unnecessary directives from spec file

Comment 5 Miroslav Spousta 2014-05-16 14:36:52 UTC
New Package SCM Request
=======================
Package Name: judy-fk
Short Description: General purpose dynamic array with fixed-length keys
Upstream URL: http://github.com/qiq/judy-fk
Owners: qiq
Branches: f19 f20 el6 epel7

Comment 6 Gwyn Ciesla 2014-05-16 15:54:58 UTC
Git done (by process-git-requests).


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