Bug#823187: RFS: icdiff/1.7.3-1

2016-05-15 Thread Sascha Steinbiss
Hi Paul, > Uploaded. Cool, thanks, and for the other hints as well. Will work on the patches right away, and let’s see what upstream does for the next version. Cheers Sascha signature.asc Description: Message signed with OpenPGP using GPGMail

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-15 Thread Paul Wise
On Fri, May 13, 2016 at 4:35 PM, Sascha Steinbiss wrote: > https://mentors.debian.net/debian/pool/main/i/icdiff/icdiff_1.8.1-1.dsc Uploaded. Some things to fix for the next upload: debian/source/include-binaries is no longer needed debian/patches/fix_version.patch needs updating, probably

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-13 Thread Sascha Steinbiss
Hi Paul, thanks for your quick reply. [...] > Interesting. Personally if I were upstream I'd drop the curl method and > go with pip but I understand it is a simple and thus useful hack. So would I, but I respect upstream's wishes and there was also a user who preferred the simple, stand-alone

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-12 Thread Paul Wise
On Thu, 2016-05-12 at 22:37 +0100, Sascha Steinbiss wrote: > Upstream is still unsure how to address this problem properly given > that he wants to retain a specific structure to his code. There have > been a few suggestions and PRs from myself and others, but they are > apparently not quite what

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-12 Thread Sascha Steinbiss
Hi Paul, some quick news regarding icdiff... […] >>> AFAICS icdiff is currently compatible with both Python 2 and Python 3. >> >> Excellent. In case upstream doesn't want to switch to using Python 3 in >> the script, I would recommend the Debian version do that. > > OK, I have already done

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-03 Thread Sascha Steinbiss
Hi Paul, >> I see. Unfortunately the copy of this file that upstream ships does >> not include a license; however a newer version does. >> I have excluded the old version from 1.7.6 using d/copyright's Files- >> Excluded, and then patched the code to use the newer version I >> distribute myself

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-02 Thread Paul Wise
On Mon, 2016-05-02 at 20:49 +0100, Sascha Steinbiss wrote: > ... Great! > I see. Unfortunately the copy of this file that upstream ships does > not include a license; however a newer version does. > I have excluded the old version from 1.7.6 using d/copyright's Files- > Excluded, and then

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-02 Thread Sascha Steinbiss
Hi Jakub, > The syntax is: > > .so foobar.1 > > Note that dh_installman replaces manpages that contain only a .so with > symlinks; so alternatively you could make the symlink yourself. Excellent, I will do this then. Many thanks for the quick help! Cheers Sascha

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-02 Thread Jakub Wilk
* Paul Wise , 2016-05-02, 14:31: I can't find the syntax right now, but there is a way for manual pages to redirect from one to the other instead of duplicating them. The syntax is: .so foobar.1 Note that dh_installman replaces manpages that contain only a .so with

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-02 Thread Sascha Steinbiss
Hi Paul thanks for the review. Let’s see what I can do. > These need to be fixed before I upload this: > >> This NEW upload closes RFP #818245. > > Any particular reason you haven't retitled it to ITP and set yourself > as the owner? Done. > Since this is a fork of part of the Python stdlib,

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-02 Thread Paul Wise
Control: tags -1 + moreinfo On Mon, May 2, 2016 at 6:14 AM, Sascha Steinbiss wrote: > Alternatively, one can download the package with dget using this command: > > dget -x > https://mentors.debian.net/debian/pool/main/i/icdiff/icdiff_1.7.3-1.dsc Here is a review: These need to be fixed

Bug#823187: RFS: icdiff/1.7.3-1

2016-05-01 Thread Sascha Steinbiss
Package: sponsorship-requests Severity: wishlist Dear mentors, I am looking for a sponsor for my package “icdiff" * Package name : icdiff Version : 1.7.3-1 Upstream Author : Jeff Kaufman * URL : http://www.jefftk.com/icdiff * License : Python Section : utils It builds