Bug 506476

Summary: Package Review: rest - A library for access to RESTful web services
Product: [Fedora] Fedora Reporter: Peter Robinson <pbrobinson>
Component: Package ReviewAssignee: Yanko Kaneti <yaneti>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: bnocera, fedora-package-review, notting, yaneti
Target Milestone: ---Flags: yaneti: fedora-review+
j: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-06-24 20:34:25 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 506450    
Bug Blocks: 506446, 506486, 506833    

Description Peter Robinson 2009-06-17 13:25:02 UTC
SPEC: http://pbrobinson.fedorapeople.org/rest.spec
SRPM: http://pbrobinson.fedorapeople.org/rest-0.3-1.fc11.src.rpm

This library was designed to make it easier to access web services that
claim to be "RESTful". A RESTful service should have urls that represent
remote objects, which methods can then be called on. The majority of services
don't actually adhere to this strict definition. Instead, their RESTful end
point usually has an API that is just simpler to use compared to other types
of APIs they may support (XML-RPC, for instance). It is this kind of API that
this library is attempting to support.

Comment 1 Yanko Kaneti 2009-06-17 15:20:12 UTC
After going through the MUST list:

rpmlint:
rest.src: W: invalid-license LGPLv2.1

should be License: LGPLv2



Some cosmetic things:

BuildRequires: gtk-doc
is not really necessary , the tarball has that already built.

You can use 
%configure --disable-static
and spare one rm line later.


The source tarball matches upstream, the package builds in rawhide i386 and x86_64 mock.
After the license is is fixed this package looks fit to be APPROVED.

Comment 2 Peter Robinson 2009-06-17 15:51:53 UTC
I'll update the above.

Comment 3 Peter Robinson 2009-06-18 11:09:01 UTC
New Package CVS Request
=======================
Package Name: rest
Short Description: A library for access to RESTful web services 
Owners: pbrobinson
Branches: F-11 F-10
InitialCC:

Comment 4 Jason Tibbitts 2009-06-18 20:30:46 UTC
CVS done.

Comment 5 Peter Robinson 2009-06-18 22:20:53 UTC
built in rawhide

Comment 6 Peter Robinson 2009-06-24 20:34:25 UTC
Done. Thanks for the review!

Comment 7 Peter Robinson 2010-01-22 21:26:53 UTC
Package Change Request
======================
Package Name: rest
New Branches: EL-4 EL-5
Owners: pbrobinson

Comment 8 Jason Tibbitts 2010-01-24 19:58:26 UTC
CVS done.