Bug#1080356: RFS: gxkb/0.9.6+ds-1 -- X11 keyboard indicator and switcher

[Pierre Gruet]

Hi Mateusz,

Le 20/09/2024 à 20:38, Mateusz Łukasik a écrit :

Hi Pierre,

On 17.09.2024 00:37, Pierre Gruet wrote:

Hi Mateusz,


> Mateusz,
>
> Preamble...
>

> to the Debian project.
>

> submitters to Debian mentors inorder to improve their packages prior to

> review.
>
> Review...
>
> 1. Build:
>
> * pbuilder [1]: Good
> * sbuild [2]: Good
>
> 2. Lintian [3]: Good
>
> 3. Licenses [4]: Issue
>

> en: Versions: recon 1.18 check 3.3.9-1
>
> Parsing Source Tree ....
> Reading copyright ....
> Running licensecheck ....
>
> d/copyright | licensecheck
>
> GPL-2+ | FSFAP doc/INSTALL
>


In addition to what Phil pointed out (thanks to you Phil!):

- the licensing information of doc/INSTALL is missing in d/copyright
- the licensing information of src/Makefile.am is missing in d.copyright
- Alexander Iliev is holding copyright of many files, not only src/gxkb.c

>
> 4. Watch file [uscan --force-download]: Good
>
> 5. Build Twice [sudo pbuilder build --twice <package>.dsc]: Good
>
> 6. Reproducible builds [5]: Good
>
> 7. Install [No previous installs]: Good
>
> 8. Upgrade [Over previous installs if any]: Good
>



Cheers,

On Mon, 16 Sep 2024 02:57:12 +0100 Phil Wyett <philip.wy...@kathenas.org> wrote: > Thank you for taking the time to prepare this package and your contribution > The review below is for assistance. This review is offered to help package > possible sponsorship into Debian. There is no obligation on behalf of the > submitter to make any alterations based upon information provided in the > philwyett@ks-tarkin:~/Development/builder/debian/mentoring/ gxkb-0.9.6+ds$ lrc > File has its own license header, so should really be added to 'd/ copyright'. Otherwise the package is in good shape and I would consider sponsoring it after the d/copyright changes listed above have been made.

All missing fields in d/copyright is fixed in last upload.

Thanks!

However, when I execute
$ dget https://mentors.debian.net/debian/pool/main/g/gxkb/gxkb_0.9.6-1.dsc
$ dpkg-source -x gxkb_0.9.6-1.dsc



Many thanks again,

--
Pierre

then the date in d/changelog is September 2nd, 2024 which does not fit the one in your git repo, and also the d/copyright entries you added are not there. Could you please check the files that were uploaded yesterday ?

OpenPGP_signature.asc
Description: OpenPGP digital signature