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

Bug#823187: RFS: icdiff/1.7.3-1



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 that in my version here and will push it with the next upload.

Done.

>>> Please let me know what you think.
>> 
>> How about packaging 1.8.1 and then I'll upload it?
> 
> 1.8.1 currently FTBFS due to a setuptools-related change that was also
> introduced in this version. Currently discussing this with upstream to
> sort it out. I will ping the list again if there has been progress.

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 upstream is going for. For more information see https://github.com/jeffkaufman/icdiff/issues/68 and https://github.com/jeffkaufman/icdiff/pull/79. 
For simplicity’s sake in the meantime, I have skipped using setuptools in the Debian package and just install the single stand-alone script manually, an option that upstream explicitly wants to support anyway. I hope that is acceptable for a first upload as it does’t change much in the installed version — I will re-enable support for setuptools again of course as soon as upstream has sorted out these issues.

New upload is here: https://mentors.debian.net/debian/pool/main/i/icdiff/icdiff_1.8.1-1.dsc

Cheers
Sascha

Reply to: