Hi Vincent,
I've saved a copy of git-archive-all in debian directory and updated
copyright accordingly.
The package has been reuploaded to mentors.
Thank you!
Best regards,
Andy
On Fri, Sep 18, 2015 at 3:57 PM, Vincent Bernat <ber...@debian.org> wrote:
> ❦ 17 septembre 2015 19:17 +0800, Andy Li <a...@onthewings.net> :
>
> > I've revised the package and reuploaded.
>
> In d/rules, get-orig-source is using "curl | python". I am a bit
> uncomfortable with that. I had a quick look at this Python script and it
> should be replacable by something like that:
>
> #+BEGIN_SRC sh
> (
> git ls-files
> git submodule --quiet foreach \
> --recursive 'git ls-files --with-tree=${sha1} | sed
> s+^+${path}/+'
> ) | while read f; do
> [[ -d $f ]] || echo $f
> done | tar -zcf orig.tar.gz -T -
> #+END_SRC
>
> If not, it would be better to retrieve the script and place it in the
> debian/ directory (and update d/copyright).
>
> > I'm not very sure about the lintian hardening warnings. Lintian didn't
> > report those to me when I build the package, maybe because I have a
> > different environment than yours? I've added a lintian-override file
> > anyway.
>
> I am using "lintian -viI". Not everything needs an override (notably
> stuff with "I"). Your override worked fine.
>
> I don't see any additional problem with your packaging.
> --
> Debian package sponsoring guidelines:
> http://vincent.bernat.im/en/debian-package-sponsoring.html
>
