Bug#803315: debsnap: use JSON::PP

Jakub Wilk jwilk at debian.org
Sun Nov 1 17:35:25 UTC 2015


* RjY <rjy at users.sourceforge.net>, 2015-10-28, 17:23:
>debsnap seems to work perfectly well with JSON::PP, part of perl core 
>since 5.14, according to its manual page. So it might be nice to use it 
>instead of requiring the extra dependency?

I'd welcome this change too. Not that long ago I wanted to use debsnap 
on a machine that didn't have libjson-perl installed, and it sucked.

>I think running a bit slower (by using pure perl JSON implementation 
>instead of optimized C one) is preferable to not running at all :)

Yeah, I don't think we should be worried about JSON parser speed here.

I've attached a more complete patch. It gets rid of now unused error 
handling code, and updates debian/control.

-- 
Jakub Wilk
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 803315.diff
Type: text/x-diff
Size: 1682 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/devscripts-devel/attachments/20151101/527ac31e/attachment.diff>


More information about the devscripts-devel mailing list