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

Re: nthash is fine - please review two other packages. :-)



On 2021-03-27 02:43, Andreas Tille wrote:
> On Sat, Mar 27, 2021 at 02:33:03AM +0530, Nilesh Patra wrote:
>> > Please check the probably
>> > unneeded empty file debian/tests/test1.  Feel free to upload yourself.
>> > It looks good from my point.
>> I uploaded, thank you for the review! :-)
>> > I wonder whether you (or whoever likes to!) like to have a look at
>> > python-lineages[2].  I did not found any license information
>> > specifically I'm not sure about the included font.
>>
>> Well, I do see a license on the g/h repo  here: https://github.com/cov-lineages/lineages/blob/master/LICENSE.md
>> They probably did not include it in the tarball.
>>
>> But there is a bigger problem to solve here -- the license is CC BY-NC-SA which makes it non-DFSG compliant :-/
>> We need to talk to and convince upstream there. To be fair, I've first time seen code being licensed under creative commons so probably we could suggest a better one maybe MIT/Expat?
> 
> Yes, definitely.  I'd be happy if you'd volunteer to discuss - missing
> the file inside the tarball is an issue anyway.

Done: https://github.com/cov-lineages/lineages/issues/17
Feel free to add anything you think would be missing :-)

>> Hmm, did you pick the wrong one by mistake?
>> Quite literally, it has "1" file: https://salsa.debian.org/med-team/python-pangolearn/-/blob/master/pangoLEARN/__init__.py
>> inside it which contains code, that too this:
>>
>> _program = "pangoLEARN"
>> __version__ = "2021-03-16"
>>
>>
>> which does not seem to do anything useful. So I'm quite confused as to what this really does.
>> Steffen might have a better answer for this one -- I do not really know what this package does.
> 
> I also have no idea.  Its a precondition for pangolin and I simply
> expect setup.py doing the right thing that the resulting executable
> in /usr/bin simply is not failing.

Right, but it simply seems to call something which does not exist. I
think I'll wait for Steffen's reply on this

Nilesh


Reply to: