RFS: flickcurl

2007-08-05 Thread Kumar Appaiah
Dear Debian Mentors,

I would request a mentor to have a look at the following package and
consider sponsoring it:

http://mentors.debian.net/debian/pool/main/f/flickcurl/flickcurl_0.11-1.dsc

It builds the following packages:
flickcurl - call Flickr API from command line
flickrdf - generate RDF triples from a Flickr photo
libflickcurl-dev - C library for accessing the Flickr API - development files
libflickcurl0 - C library for accessing the Flickr API
libflickcurl0-dbg - C library for accessing the Flickr API

From my ITP (bug #436065):
 Package name: flickcurl
  Version : 0.11
  Upstream Author : Dave Beckett
* URL : http://librdf.org/flickcurl/
* License : LGPL 2.1 / GPL 2 /Apache 2.0 (I have chosen GPL)
  Programming Lang: C
  Description : C library for the Flickr API
Flickcurl is a C library for the Flickr API, handling creating the
requests, signing, token management, calling the API, marshalling
request parameters and decoding responses. It uses libcurl to call the
REST web service and libxml2 to manipulate the XML responses. The
current version supports part of the API, primarily the functions for
reading photo, people and tags description, uploading photos, changing
tags and comments.

Thanks in advance.

Kumar
-- 
Kumar Appaiah,
458, Jamuna Hostel,
Indian Institute of Technology Madras,
Chennai - 600 036


signature.asc
Description: Digital signature


Re: RFS: flickcurl

2007-08-05 Thread Mario Iseli
On Sun, Aug 05, 2007 at 09:53:19PM +0530, Kumar Appaiah wrote:
 Dear Debian Mentors,

Hi,

  Package name: flickcurl
   Version : 0.11
   Upstream Author : Dave Beckett
 * URL : http://librdf.org/flickcurl/
 * License : LGPL 2.1 / GPL 2 /Apache 2.0 (I have chosen GPL)
   Programming Lang: C
   Description : C library for the Flickr API

Ok, here my comments about it:

debian/control:
Well done, really... :)

debian/copyright:
You have some useless whitespaces mostly at the end of the lines. Then
ehm: License: LGPL 2.1 / GPL 2 / Apache 2.0
What should that mean? If you have several licenses in your package you
have to list which files and which author did what, for example have a
look at
http://packages.debian.org/changelogs/pool/main/m/mcabber/mcabber_0.9.3-1/mcabber.copyright
how to do that. You really have to go through all the source files and
add every single copyright to this file, that's important.

debian/flickcurl.1:
Nice manpage, but please remove the comment lines at the beginning since
they are from the template and not used.

debian/flickcurl-config.1:
Well done again...

debian/flickcurl.manpages:
- ./debian/flickcurl.1
+ debian/flickcurl.1

debian/flickrdf.docs:
Useless empty line at the end of the file.

debian/rules:
Also some useless whitespaces at the end of the line, if you use vim
please add the following to your vimrc and then you will see what i
mean:
syntax on
highlight WhitespaceEOL ctermbg=red guibg=red
match WhitespaceEOL /\s\+$/

debian/watch:
Well done :)


So, hmm yes, I'm generally interested to sponsor your package, please
register it on sponsors.debian.net, I use that to track my sponsoring
and get in contact with me if you have fixed the things from above.

Thank you and regards,

-- 
  .''`. Mario Iseli [EMAIL PROTECTED]
 : :'  :Debian GNU/Linux developer
 `. `'`
   `-  Debian - when you have better things to do than fixing a system


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Re: RFS: flickcurl

2007-08-05 Thread Kumar Appaiah
Dear Mario,

First of all, thanks for the quick review.

On Sun, Aug 05, 2007 at 06:43:50PM +0200, Mario Iseli wrote:
 debian/control:
 Well done, really... :)

Well, I must thank the last mentor who guided me through a library
process (Neil codehelp Williams). So, this was pretty all right.

 debian/copyright:
 You have some useless whitespaces mostly at the end of the lines. Then
 ehm: License: LGPL 2.1 / GPL 2 / Apache 2.0
 What should that mean? If you have several licenses in your package you
 have to list which files and which author did what, for example have a
 look at
 http://packages.debian.org/changelogs/pool/main/m/mcabber/mcabber_0.9.3-1/mcabber.copyright
 how to do that. You really have to go through all the source files and
 add every single copyright to this file, that's important.

OK, so here's the deal. The author gives you the option of choosing
from among GPL 2+, LGPL 2.1 or the Apache license. And, I *have*
checked every file. For example, see this:

find |egrep \.(h|c)$|xargs grep (GPL)

Compare that to find|egrep \.(h|c)$. That shows that example.c
(which is an example) and flickcurl_getopt.h may not be GPL'd. So, I
have mentioned flickcurl_getopt.h as being public domain, and all
other files as GPL.

And since the COPYING file is included, I am going ahead with the
assumption that it is GPL'd. Besides, it'd be a pity if a 50 line C
example program is discarded!

 debian/flickcurl.1:
 Nice manpage, but please remove the comment lines at the beginning since
 they are from the template and not used.

Done.

 debian/flickcurl-config.1:
 Well done again...

Thanks.

 debian/flickcurl.manpages:
 - ./debian/flickcurl.1
 + debian/flickcurl.1

OK. Done.

 debian/flickrdf.docs:
 Useless empty line at the end of the file.

Removed.

 debian/rules:
 Also some useless whitespaces at the end of the line, if you use vim
 please add the following to your vimrc and then you will see what i
 mean:
 syntax on
 highlight WhitespaceEOL ctermbg=red guibg=red
 match WhitespaceEOL /\s\+$/

I think I've done it right now.

 debian/watch:
 Well done :)

Thanks!

 So, hmm yes, I'm generally interested to sponsor your package, please
 register it on sponsors.debian.net, I use that to track my sponsoring
 and get in contact with me if you have fixed the things from above.

Well, thanks a lot! I have registered the package on
Sponsors.d.net. Please check it out, and many thanks for the review
and upload offer! :-)

Kumar
-- 
Kumar Appaiah,
458, Jamuna Hostel,
Indian Institute of Technology Madras,
Chennai - 600 036


signature.asc
Description: Digital signature


Re: RFS: flickcurl

2007-08-05 Thread Kumar Appaiah
On Sun, Aug 05, 2007 at 11:08:49PM +0530, Kumar Appaiah wrote:
  debian/copyright:
  You have some useless whitespaces mostly at the end of the lines. Then
  ehm: License: LGPL 2.1 / GPL 2 / Apache 2.0
  What should that mean? If you have several licenses in your package you
  have to list which files and which author did what, for example have a
  look at
  http://packages.debian.org/changelogs/pool/main/m/mcabber/mcabber_0.9.3-1/mcabber.copyright
  how to do that. You really have to go through all the source files and
  add every single copyright to this file, that's important.
 
 OK, so here's the deal. The author gives you the option of choosing
 from among GPL 2+, LGPL 2.1 or the Apache license. And, I *have*
 checked every file. For example, see this:
 
 find |egrep \.(h|c)$|xargs grep (GPL)

Oops! One more miss! getopt.c is also public domain!

find |egrep \.(h|c)$|xargs grep -L (GPL) gives getopt.c,
flickcurl_getopt.h and example.c

I am reuploading with the modification in the copyright file. Sorry.

Kumar
-- 
Kumar Appaiah,
458, Jamuna Hostel,
Indian Institute of Technology Madras,
Chennai - 600 036


signature.asc
Description: Digital signature


Re: RFS: flickcurl

2007-08-05 Thread Kumar Appaiah
On Sun, Aug 05, 2007 at 11:16:12PM +0530, Kumar Appaiah wrote:
 Oops! One more miss! getopt.c is also public domain!
 
 find |egrep \.(h|c)$|xargs grep -L (GPL) gives getopt.c,
 flickcurl_getopt.h and example.c
 
 I am reuploading with the modification in the copyright file. Sorry.

Done! Copyright now reads this
copyright
License for flickcurl_getopt.h and getopt.c: Public domain

License for all other files:
gpl-stuff/
/copyright

And uploaded to mentors. Hope it's OK now.

http://mentors.debian.net/debian/pool/main/f/flickcurl/flickcurl_0.11-1.dsc

Thanks!

Kumar
-- 
Kumar Appaiah,
458, Jamuna Hostel,
Indian Institute of Technology Madras,
Chennai - 600 036


signature.asc
Description: Digital signature