Bug 468414 - Review Request: hamcrest - Hamcrest matcher object framework
Summary: Review Request: hamcrest - Hamcrest matcher object framework
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Permaine Cheung
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-10-24 16:41 UTC by David Walluck
Modified: 2008-11-21 16:27 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-11-21 16:27:24 UTC
Type: ---
Embargoed:
pcheung: fedora-review+
dennis: fedora-cvs+


Attachments (Terms of Use)
build log (36.93 KB, text/x-log)
2008-10-24 18:48 UTC, Permaine Cheung
no flags Details

Description David Walluck 2008-10-24 16:41:33 UTC
Spec URL: <http://zarb.org/~david/hamcrest.spec>
SRPM URL: <http://zarb.org/~david/hamcrest-1.1-2.jpp5.src.rpm>
Description :
Provides a library of matcher objects (also known as
constraints or predicates) allowing 'match' rules to
be defined declaratively, to be used in other frameworks.
Typical scenarios include testing frameworks, mocking
libraries and UI validation rules.

Comment 1 David Walluck 2008-10-24 16:42:17 UTC
This package is required by junit4 4.4 and later.

Comment 2 Permaine Cheung 2008-10-24 17:03:21 UTC
I'll take this one.

Comment 3 Permaine Cheung 2008-10-24 18:48:06 UTC
Created attachment 321455 [details]
build log

Comment 4 Permaine Cheung 2008-10-24 18:49:01 UTC
I can't build this in mock (fedora-devel-i386), please see attached build.log.

Comment 5 Permaine Cheung 2008-10-24 21:48:48 UTC
Naming: OK
Licensing: OK
No inclusion of pre-built binaries or libraries: OK
Filesystem layout: OK
No Packager, Vendor tag: OK
Summary tag not ending with a period: OK
Source tag: OK (md5sum matches)
Buildroot tag: OK
%install begins with removing buildroot: OK
Changelog: OK

Jar file naming: OK
Directory structure: OK
BuildRequires and Requires:  OK

Comment 6 Permaine Cheung 2008-10-24 22:23:34 UTC
Running rpmlint against the mock built rpms:
rpmlint /var/lib/mock/fedora-development-i386/result/*rpm
hamcrest.i386: W: non-conffile-in-etc /etc/maven/fragments/hamcrest
hamcrest-demo.i386: W: no-documentation

rpm -qp --provides hamcrest-1.1-5.1.fc10.i386.rpm
all-1.1.jar.so  
hamcrest = 0:1.1-5.1.fc10
hamcrest(x86-32) = 0:1.1-5.1.fc10

Is the Provides for all-1.1.jar.so correct?

Comment 7 David Walluck 2008-10-24 22:44:56 UTC
hamcrest.i386: W: non-conffile-in-etc

This is a bug in maven, as this file is not really a config file.

hamcrest-demo.i386: W: no-documentation

The demo depends on the main package, which contains the license. Otherwise, there is no other documentation.

The all-1.1.jar.so was added automatically. It seems kind of weird.

Comment 8 Permaine Cheung 2008-10-25 02:51:36 UTC
Links for the spec file and SRPM that builds successfully: http://zarb.org/~david/hamcrest.spec
http://zarb.org/~david/hamcrest-1.1-5.1.jpp5.src.rpm

APPROVED

Comment 9 David Walluck 2008-10-25 15:23:36 UTC
New Package CVS Request
=======================
Package Name: hamcrest
Short Description: Hamcrest matcher object framework
Owners: dwalluck
Branches: F-10
InitialCC:

Comment 10 Dennis Gilmore 2008-10-27 04:13:51 UTC
CVS Done


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