Bug 506476 - Package Review: rest - A library for access to RESTful web services
Summary: Package Review: rest - A library for access to RESTful web services
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Yanko Kaneti
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 506450
Blocks: FedoraMoblin20 506486 506833
TreeView+ depends on / blocked
 
Reported: 2009-06-17 13:25 UTC by Peter Robinson
Modified: 2010-01-24 19:58 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-06-24 20:34:25 UTC
Type: ---
Embargoed:
yaneti: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

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.


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