[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: sxiv



Hi Daniel,

On Thu, 2011-06-30 at 20:37 -0500, Daniel Echeverry wrote:
> I am looking for a sponsor for my package "sxiv".
> 
> 
> * Package name    : sxiv
>   Version         : 0.8.2-1
>   Upstream Author : Bert Münnich <muennich@informatik.hu-berlin.de>
> * URL             : https://github.com/muennich/sxiv
> * License         : GPL2+
>   Section         : graphics
> 
> 
> It builds these binary packages:
> sxiv       - simple X image viewer
> 
> The package appears to be lintian clean.
> The upload would fix these bugs: 631728
> 
> My motivation for maintaining this package is: 

> Always I wanted a lightweight viewer for my debian, I recently met
> sxiv and I liked for its ease and simplicity. I hope to keep the
> package and provide for this in debian for a long time

I see you've found "feh" already which I would have proposed as very
slim image viewer. ;-)


> The package can be found on mentors.debian.net:
> - URL: http://mentors.debian.net/debian/pool/main/s/sxiv
> - Source repository: deb-src http://mentors.debian.net/debian unstable
> main contrib non-free
> - dget
> http://mentors.debian.net/debian/pool/main/s/sxiv/sxiv_0.8.2-1.dsc

Comments are:

- all upstream copyright still has old FSF address

- Your installation of the manpage does break upstream's logic to write
the version number into the manpage. I would consider installing the
manpages like upstream proposes the preferred approach. So IMHO your
patch should be removed (and the sxiv.manpages file in debian/)

- The upstream manpage however still has two hyphen-used-as-minus-sign
issues:
usr/share/man/man1/sxiv.1.gz:79
usr/share/man/man1/sxiv.1.gz:239



> I would be glad if someone uploaded this package for me.

I've taken the liberty to remove your patch, build, sign and upload your
package.

For your info the package is at http://people.debian.org/~kilian/sxiv/
as was uploaded.

Thanks for your work!

-- 
Best regards,
Kilian

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: