Bug 506476

Summary:	Package Review: rest - A library for access to RESTful web services
Product:	[Fedora] Fedora	Reporter:	Peter Robinson <pbrobinson>
Component:	Package Review	Assignee:	Yanko Kaneti <yaneti>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	low
Version:	rawhide	CC:	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.