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]: Issue I: libaudtag3t64: hardening-no-fortify-functions [usr/lib/x86_64-linux- gnu/libaudtag.so.3.0.0] N: N: This package provides an ELF binary that lacks the use of fortified libc N: functions. Either there are no potentially unfortified functions called by N: any routines, all unfortified calls have already been fully validated at N: compile-time, or the package was not built with the default Debian N: compiler flags defined by dpkg-buildflags. If built using dpkg- buildflags N: directly, be sure to import CPPFLAGS. N: N: NB: Due to false-positives, Lintian ignores some unprotected functions N: (e.g. memcpy). N: N: Please refer to https://wiki.debian.org/Hardening and Bug#673112 for N: details. N: N: Visibility: info N: Show-Always: no N: Check: binaries/hardening I: libaudcore5t64: unused-override package-name-doesnt-match-sonames libaudcore5 [usr/share/lintian/overrides/libaudcore5t64:1] N: N: Your package specifies the named override but there were no tags that N: could have been silenced by it. N: N: Maybe you fixed an underlying condition but forgot to remove the override. N: It is also possible that the Lintian maintainers fixed a false positive. N: N: If the override is now unused, please remove it. N: N: This tag is similar to mismatched-override except there a tag could have N: been silenced if the context had matched. N: N: Sometimes, overrides end up not being used because a tag appears only on N: some architectures. In that case, overrides can be equipped with an N: architecture qualifier. N: N: Please refer to Architecture specific overrides (Section 2.4.3) in the N: Lintian User's Manual for details. N: N: Visibility: info N: Show-Always: yes N: Check: lintian N: I: libaudtag3t64: unused-override package-name-doesnt-match-sonames libaudtag3 [usr/share/lintian/overrides/libaudtag3t64:1] P: audacious source: trailing-whitespace [debian/changelog:12] N: N: This file contains lines with trailing whitespace characters. N: N: Whilst often harmless and unsightly, such extra whitespaces can also cause N: tools to interpret the whitespace characters literally. The tool diff(1) N: does not like them, either. They are best avoided. N: N: Some of these problems can be hard to track down. N: N: Whitespace at the end of lines may be removed with the following: N: N: $ sed -i -e 's@[[:space:]]*$@@g' debian/control debian/changelog N: N: If you use Emacs, you can also use "M-x wh-cl" (whitespace-cleanup). N: N: However, if you wish to only remove trailing spaces and leave trailing N: tabs (eg. for Makefiles), you can use the following code snippet: N: N: $ sed -i -e 's@[ ]*$@@g' debian/rules N: N: To remove empty lines from the end of a file, you can use: N: N: $ sed -i -e :a -e '/^\n*$/{$d;N;};/\n$/ba' debian/rules N: N: Visibility: pedantic N: Show-Always: no N: Check: debian/trailing-whitespace N: Renamed from: file-contains-trailing-whitespace 3. Licenses [4]: Issue philwyett@ks-tarkin:~/Development/builder/debian/mentoring/audacious-4.4$ lrc : Versions: recon 1.16 check 3.3.9-1 Parsing Source Tree .... Reading copyright .... Running licensecheck .... d/copyright | licensecheck BSD-2-clause | FSFAP INSTALL 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 Summary... I believe audacious 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 * 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 -- "I play the game for the game’s own sake" Arthur Conan Doyle - The Adventure of the Bruce-Partington Plans -- Buy Me A Coffee: https://buymeacoffee.com/kathenasorg Internet Relay Chat (IRC): kathenas Matrix: #kathenas:matrix.org Website: https://kathenas.org Instagram: https://instagram.com/kathenasorg/ Threads: https://www.threads.net/@kathenasorg --
signature.asc
Description: This is a digitally signed message part
