Details about package binutils-avr
Package uploads
Upload #2
Information
Changelog
binutils-avr (2.43.1-1) unstable; urgency=low
.
* Switch to mainstream binutils (Closes: #855849)
* Fix fail to build source after build (Closes: #1044001)
* Switch over to dh sequencer
Comments
-
Steve,
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]: Information only
Running lintian...
N:
I: binutils-avr: typo-in-manual-page ambigious ambiguous [usr/share/man/man1/avr-addr2line.1.gz:134]
N:
N: Lintian found a spelling error in a manual page. Lintian has a list of
N: common misspellings that it looks for. It does not have a dictionary like
N: a spelling checker does.
N:
N: If the string containing the spelling error is translated with the help of
N: gettext (with the help of po4a, for example) or a similar tool, please fix
N: the error in the translations as well as the English text to avoid making
N: the translations fuzzy. With gettext, for example, this means you should
N: also fix the spelling mistake in the corresponding msgids in the *.po
N: files.
N:
N: Visibility: info
N: Show-Always: no
N: Check: documentation/manual
N: Renamed from: spelling-error-in-manpage
N:
N:
I: binutils-avr: typo-in-manual-page complier compiler [usr/share/man/man1/avr-ld.1.gz:827]
N:
I: binutils-avr: typo-in-manual-page supressed suppressed [usr/share/man/man1/avr-objdump.1.gz:267]
N:
I: binutils-avr: typo-in-manual-page targetted targeted [usr/share/man/man1/avr-ld.1.gz:2155]
N:
P: binutils-avr: repeated-path-segment lib [usr/lib/avr/lib/]
N:
N: The file is installed into a location that repeats the given path segment.
N: An example would be /usr/lib/lib or /usr/share/myprogram/share.
N:
N: More often than not this is unintended.
N:
N: Please refer to Bug#950052 and Bug#950027 for details.
N:
N: Visibility: pedantic
N: Show-Always: no
N: Check: files/hierarchy/path-segments
N:
N:
P: binutils-avr: repeated-path-segment lib [usr/lib/avr/x86_64-linux-gnu/lib/]
N:
P: binutils-avr: spelling-error-in-readme-debian diffrent different [usr/share/doc/binutils-avr/README.Debian]
N:
N: Lintian found a spelling error in the README.Debian file. Lintian has a
N: list of common misspellings that it looks for. It does not have a
N: dictionary like a spelling checker does.
N:
N: Visibility: pedantic
N: Show-Always: no
N: Check: debian/readme
N:
I: Lintian run was successful.
3. Licenses [4]: Good
4. Watch file [uscan --force-download]: Issue
philwyett@ks-tarkin:~/Development/builder/debian/binutils-avr-2.43.1$ uscan --force-download
gpgv: Signature made Sat 17 Aug 2024 18:04:05 BST
gpgv: using RSA key 3A24BC1E8FB409FA9F14371813FCEF89DD9E3C4F
gpgv: Can't check signature: No public key
uscan die: OpenPGP signature did not verify. at /usr/share/perl5/Devscripts/Uscan/Output.pm line 77.
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 binutils-avr is not yet ready for sponsorship at this time. Could the contributor rectify one of more of the raised issues.
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
Needs work
Phil Wyett at Oct. 21, 2024, 3:59 p.m.
Upload #1
Information
Changelog
binutils-avr (2.43.1-1) unstable; urgency=low
.
* Switch to mainstream binutils (Closes: #855849)
* Fix fail to build source after build (Closes: #1044001)
* Switch over to dh sequencer
Comments
-
Steve,
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]: Warnings / Information
Running lintian...
N:
W: waycheck: no-manual-page [usr/bin/waycheck]
N:
N: Each binary in /usr/bin, /usr/sbin, /bin, /sbin or /usr/games should have
N: a manual page
N:
N: Note that though the man program has the capability to check for several
N: program names in the NAMES section, each of these programs should have its
N: own manual page (a symbolic link to the appropriate manual page is
N: sufficient) because other manual page viewers such as xman or tkman don't
N: support this.
N:
N: If the name of the manual page differs from the binary by case, man may be
N: able to find it anyway; however, it is still best practice to match the
N: exact capitalization of the executable in the manual page.
N:
N: If the manual pages are provided by another package on which this package
N: depends, Lintian may not be able to determine that manual pages are
N: available. In this case, after confirming that all binaries do have manual
N: pages after this package and its dependencies are installed, please add a
N: Lintian override.
N:
N: Please refer to Manual pages (Section 12.1) in the Debian Policy Manual
N: for details.
N:
N: Visibility: warning
N: Show-Always: no
N: Check: documentation/manual
N: Renamed from: binary-without-manpage
N:
N:
I: waycheck: desktop-entry-lacks-keywords-entry [usr/share/applications/dev.serebit.Waycheck.desktop]
N:
N: This .desktop file is either missing a Keywords entry, or it does not
N: contain keywords above and beyond those already present in the Name or
N: GenericName entries.
N:
N: The Keywords field is intended to show keywords relevant for a .desktop
N: file.
N:
N: Desktop files are organized in key-value pairs and are similar to INI
N: files.
N:
N: The desktop-file-validate tool in the desktop-file-utils package may be
N: useful when checking the syntax of desktop entries.
N:
N: Please refer to
N: https://specifications.freedesktop.org/desktop-entry-spec/latest/ar01s06.html,
N: Bug#693918, and
N: https://wiki.gnome.org/Initiatives/GnomeGoals/DesktopFileKeywords for
N: details.
N:
N: Visibility: info
N: Show-Always: no
N: Check: menu-format
N:
N:
I: waycheck: spelling-error-in-binary oter other [usr/bin/waycheck:1]
N:
N: Lintian found a spelling error in the given binary. Lintian has a list of
N: common misspellings that it looks for. It does not have a dictionary like
N: a spelling checker does.
N:
N: If the string containing the spelling error is translated with the help of
N: gettext or a similar tool, please fix the error in the translations as
N: well as the English text to avoid making the translations fuzzy. With
N: gettext, for example, this means you should also fix the spelling mistake
N: in the corresponding msgids in the *.po files.
N:
N: You can often find the word in the source code by running:
N:
N: grep -rw <word> <source-tree>
N:
N: This tag may produce false positives for words that contain non-ASCII
N: characters due to limitations in strings.
N:
N: Visibility: info
N: Show-Always: no
N: Check: binaries/spelling
3. Licenses [4]: Good
4. Watch file [uscan --force-download]: Issue
philwyett@ks-tarkin:~/Development/builder/debian/mentoring/binutils-avr-2.43.1$ uscan --force-download
gpgv: Signature made Sat 17 Aug 2024 18:04:05 BST
gpgv: using RSA key 3A24BC1E8FB409FA9F14371813FCEF89DD9E3C4F
gpgv: Can't check signature: No public key
uscan die: OpenPGP signature did not verify. at /usr/share/perl5/Devscripts/Uscan/Output.pm line 77.
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...
The package is in good order, with just the #4 to fix.
Lintian warnings you may wish to help address over time.
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
Needs work
Phil Wyett at Sept. 30, 2024, 6:35 a.m.
-
Your lintian run is against waycheck, not binutils-avr. I'll look into the watch file issue, but it isn't really useful as this package uses binutils-source as its "upstream" so I really need to figure out how to point watch at it.
Ready
Steve Meliza at Sept. 30, 2024, 6:46 a.m.
