Control: tags -1 -moreinfo

On 30.08.2024 01:16, Phil Wyett wrote:
Control: tags -1 +moreinfo

Mateusz,

Preamble...

Thank you for taking the time to prepare this package and your contribution
to the Debian project.

The review below is for assistance. This review is offered to help package
submitters to Debian mentors inorder to improve their packages prior to
possible sponsorship into Debian. There is no obligation on behalf of the
submitter to make any alterations based upon information provided in the
review.

Review...

1. Build:

   * pbuilder [1]: Good
   * sbuild [2]: Good

2. Lintian [3]: Good

3. Licenses [4]: Issue, verified

philwyett@ks-tarkin:~/Development/builder/debian/mentoring/smplayer-
24.5.0+ds0$ lrc -t
: Versions: recon 1.16  check 3.3.9-1

Parsing Source Tree  ....
Reading copyright    ....
Running licensecheck ....

d/copyright     | licensecheck

BSD-3-clause    | GPL-2            webserver/mongoose.h

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

Additional...

* Remove debian-orig directory.

* '+ds0-1', '0' not needed.

Summary...

I believe smplayer is not yet ready for sponsorship at this time. Could the
contributor rectify one of more of the rasied issues. Once updated to your
satisfaction and a new upload done, please remove the 'moreinfo' tag on the
Request For Sponsorship (RFS) bug report.

Regards

Phil

[1] pbuilder:

   * Command: sudo pbuilder build <PACKAGE>.dsc
   * Document: https://wiki.ubuntu.com/PbuilderHowto.
   * Document: https://wiki.debian.org/PbuilderTricks

[2] sbuild:

   * Command: sbuild <PACKAGE>.dsc
   * Document: https://wiki.kathenas.org/pmwiki.php/Kathenas/Article00000002
   * Document: https://wiki.debian.org/sbuild

[3] lintian:

   * Command: lintian -v -i -I -E --pedantic --profile debian (*.dsc,
*.changes, *.buildinfo). Each can throw up different results, so be thorough.
   * Document: https://wiki.debian.org/Lintian

[4] lrc:

   * Command: lrc -t
   * Document: https://wiki.debian.org/CopyrightReviewTools#licenserecon

[5] reprotest

   * Command: sudo reprotest --vary=-build_path,domain_host.use_sudo=1 --auto-
build <PACKAGE>.dsc -- schroot unstable-amd64-sbuild
   * Document: https://wiki.kathenas.org/pmwiki.php/Kathenas/Article00000004
   * Document: https://wiki.debian.org/ReproducibleBuilds/
   * Document: https://wiki.debian.org/ReproducibleBuilds/Howto#Newer_method

Hi Phil,

All fixed in smplayer_24.5.0+ds-1 version


--
.''`.  Mateusz Łukasik
: :' :  l0calh0st.pl
`. `'   Debian Member - mat...@linuxmint.pl
  `-    GPG: D93B 0C12 C8D0 4D7A AFBC  FA27 CCD9 1D61 11A0 6851

Reply via email to