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

Re: RFS: python3-typed-ast, hisat2



Hi Michael,

I took a look at hisat2:

 - I would add:
      Copyright 1999, N. Jesper Larsson, all rights reserved.
   from ls.h to d/copyright
 - IMHO d/rules should have a ‘clean' override that removes the man pages built there
 - The watchfile doesn’t have any active entries? The following should work:
      opts=pasv \
      ftp://ftp.ccb.jhu.edu/pub/infphilo/hisat2/downloads/hisat2-([0-9.]+)-source.zip
 - The package does not build reproducibly for me — likely because I have disorderfs enabled in my prebuilder setup, which is not the default. It might help wrapping the $(wildcard …) lines in the Makefile with $(sort …) etc. See https://reproducible-builds.org/docs/stable-inputs/.

and python3-typed-ast:

 - Lintian still warns about the (easy to fix):
      W: python3-typed-ast source: syntax-error-in-dep5-copyright line 25: Continuation line outside a paragraph (maybe line 24 should be " .”).
 - There are two license entries for ‘Files: *’ in d/copyright. From what I can see, only typed_ast/ast27.py and typed_ast/ast35.py are Python licensed, so it might be enough to set those two to License: Python, keeping the rest as Apache 2.0?
 - Since the package seems to be quite generic, it may be better maintained with/in the DPMT [1]?
   What do you think?

Cheers and thanks for working on these
Sascha

[1] https://wiki.debian.org/Teams/PythonModulesTeam


> On 17 Aug 2016, at 16:56, Michael Crusoe <michael.crusoe@gmail.com> wrote:
> 
> Thanks!
> --
> Michael R. Crusoe
> Community Engineer & Co-founder
> Common Workflow Language project
> https://impactstory.org/u/0000-0002-2961-9670
> michael.crusoe@gmail.com
> +40 720 781 765
> +1 480 627 9108

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail


Reply to: