Got it, thanks!

I am fix and update.

Could you please check for me.

On Thu, May 28, 2020, 2:54 PM Kyle Robbertze <paddatrap...@debian.org>
wrote:

> Control: tags -1 moreinfo
>
> Hi,
>
> After reviewing the package, there are a couple things I noticed:
>
> - Missing copyright info for the following files:
>   * ./m4/*
>   * ./INSTALL
>   * ./aclocal.m4
>   * ./config.rpath
>   * ./install-sh
> - The watch file is broken - it points to your packaging repo. It should
> use the upstream launchpad repo
> - The changelog should only have 'Initial packaging (Closes: #933071)'
> as the change list. The other change lines are not needed.
> - The package builds a lot of binary packages with 1 file each. Is this
> necessary? Consider combining everything into one or two packages
>
> Cheers
> Kyle
> --
>
> ⢀⣴⠾⠻⢶⣦⠀
> ⣾⠁⢠⠒⠀⣿⡁ Kyle Robbertze
> ⢿⡄⠘⠷⠚⠋⠀ Debian Developer
> ⠈⠳⣄⠀⠀⠀⠀ https://wiki.debian.org/KyleRobbertze
>

Reply via email to