Sign me up Login

Details about package pwru

Name: pwru
Uploader: Loren M. Lang <lorenl@north-winds.org> (Debian QA page)
Description: pwru - eBPF-based Linux kernel networking debugger (program)

Package uploads

Upload #2

Information

Version: 1.0.5+ds2-1
Uploaded: 2025-01-10 12:11
Source package: pwru_1.0.5+ds2-1.dsc
Distribution: unstable
Section: golang
Priority: optional
Homepage: https://github.com/cilium/pwru
Vcs-Browser: https://salsa.debian.org/go-team/packages/pwru
Vcs-Git: https://salsa.debian.org/go-team/packages/pwru.git
Closes bugs: #1063031

Changelog

 pwru (1.0.5+ds2-1) unstable; urgency=medium
 .
   * Initial release (Closes: #1063031)

QA information

Comments

  1. Hi,

Could you update to 1.0.9 of October of last year.

I would advise tracking releases in the 'd/watch' file and not tags.

Regards

Phil
    Needs work Phil Wyett at Jan. 10, 2025, 8:18 p.m. 
  2. Loren,

Review of upload: 2025-01-10 12:11

For information about the tests run, see:

https://wiki.debian.org/PhilWyett/DebianMentoring

Test 4 (sbuild): Information only

Running lintian...
N:
W: pwru: no-manual-page [usr/bin/pwru]
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: pwru source: missing-static-built-using-field-for-golang-package (in section for pwru) [debian/control:30]
N: 
N:   The stanza for a Golang installation package in the debian/control file
N:   does not include a Static-Built-Using field that contains the
N:   ${misc:Static-Built-Using} substitution variable.
N:   
N:   The dh_golang(1) build system provides the ${misc:Static-Built-Using}
N:   variable. It is used when scheduling binNMUs.
N:   
N:   Please add the following line to the relevant stanza:
N:   
N:       Static-Built-Using: ${misc:Static-Built-Using}
N: 
N:   Visibility: info
N:   Show-Always: no
N:   Check: languages/golang/built-using
N:   Renamed from: missing-built-using-field-for-golang-package
N: 
N:
I: pwru source: out-of-date-standards-version 4.6.2 (released 2022-12-17) (current is 4.7.2)
N: 
N:   The source package refers to a Standards-Version older than the one that
N:   was current at the time the package was created (according to the
N:   timestamp of the latest debian/changelog entry). Please consider updating
N:   the package to current Policy and setting this control field
N:   appropriately.
N:   
N:   If the package is already compliant with the current standards, you don't
N:   have to re-upload the package just to adjust the Standards-Version control
N:   field. However, please remember to update this field next time you upload
N:   the package.
N:   
N:   See /usr/share/doc/debian-policy/upgrading-checklist.txt.gz in the
N:   debian-policy package for a summary of changes in newer versions of
N:   Policy.
N: 
N:   Please refer to
N:   https://www.debian.org/doc/debian-policy/upgrading-checklist.html for
N:   details.
N: 
N:   Visibility: info
N:   Show-Always: no
N:   Check: fields/standards-version
N: 
N:
I: pwru source: patch-not-forwarded-upstream [debian/patches/use-external-libpcap]
N: 
N:   According to the DEP-3 headers, this patch has not been forwarded
N:   upstream.
N:   
N:   Please forward the patch and try to have it included in upstream's version
N:   control system. If the patch is not suitable for that, please mention
N:   not-needed in the Forwarded field of the patch header.
N: 
N:   Please refer to social contract item 2, Coordination with upstream
N:   developers (Section 3.1.4) in the Debian Developer's Reference, Changes to
N:   the upstream sources (Section 4.3) in the Debian Policy Manual, and
N:   Bug#755153 for details.
N: 
N:   Visibility: info
N:   Show-Always: no
N:   Check: debian/patches/dep3
N:   Renamed from: send-patch
N: 
N:
P: pwru source: trailing-whitespace [debian/control:45]
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
N: 

E: Lintian run failed (policy violation)

Test 6 (debian/watch): Information only

philwyett@ks-tarkin:~/Development/builder/debian/pwru-1.0.5+ds2$ uscan --dehs
<dehs>
Newest version of pwru on remote site is 1.0.10~pre, local version is 1.0.5
       (mangled local version is 1.0.5)
 => Newer package available from:
        => https://github.com/cilium/pwru/archive/refs/tags/v1.0.10-pre.tar.gz
Successfully repacked ../v1.0.10-pre.tar.gz as ../pwru_1.0.10~pre+ds1.orig.tar.xz, deleting 1130 files from it.
<package>pwru</package>
<debian-uversion>1.0.5+ds2</debian-uversion>
<debian-mangled-uversion>1.0.5</debian-mangled-uversion>
<upstream-version>1.0.10~pre</upstream-version>
<upstream-url>https://github.com/cilium/pwru/archive/refs/tags/v1.0.10-pre.tar.gz</upstream-url>
<status>newer package available</status>
<target>pwru_1.0.10~pre+ds1.orig.tar.xz</target>
<target-path>../pwru_1.0.10~pre+ds1.orig.tar.xz</target-path>
<messages>Successfully downloaded upstream package: v1.0.10-pre.tar.gz

</messages>
<messages>Renamed upstream package to: v1.0.10-pre.tar.gz

</messages>
</dehs>

Test 7 (licenserecon): Information only

philwyett@ks-tarkin:~/Development/builder/debian/pwru-1.0.5+ds2$ lrc
en: Versions: licenserecon '4.2'  licensecheck '3.3.9-1'

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

d/copyright      | licensecheck

BSD-2-Clause or GPL-2.0| BSD-2-clause      bpf/LICENSE.BSD-2-Clause
BSD-2-Clause or GPL-2.0| GPL-2             bpf/LICENSE.GPL-2.0

Summary
=======

A fresh review as the package has languished in Debian Mentors for far too long.

Test 4.

* Consider looking at the lintian issues raised and possibly correct the appropriate ones.

* Consider updating to the latest 'Standards-Version' in 'debian/control' to 4.7.2.

Test 6.

* The package has a newer upstream version. Please consider packaging it.

Test 7.

* Consider adding highlighted files to 'debian/lrc.config'.
    Needs work Phil Wyett at May 10, 2025, 7:05 a.m.

Upload #1

Information

Version: 1.0.5+ds2-1
Uploaded: 2025-01-02 01:56
Source package: pwru_1.0.5+ds2-1.dsc
Distribution: unstable
Section: golang
Priority: optional
Homepage: https://github.com/cilium/pwru
Vcs-Browser: https://salsa.debian.org/go-team/packages/pwru
Vcs-Git: https://salsa.debian.org/go-team/packages/pwru.git
Closes bugs: #1063031

Changelog

 pwru (1.0.5+ds2-1) unstable; urgency=medium
 .
   * Initial release (Closes: #1063031)

QA information

Comments

  1. Loren,

Preamble...

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

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

Running lintian...
N:
W: pwru: no-manual-page [usr/bin/pwru]
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: pwru source: out-of-date-standards-version 4.6.2 (released 2022-12-17) (current is 4.7.0)
N: 
N:   The source package refers to a Standards-Version older than the one that
N:   was current at the time the package was created (according to the
N:   timestamp of the latest debian/changelog entry). Please consider updating
N:   the package to current Policy and setting this control field
N:   appropriately.
N:   
N:   If the package is already compliant with the current standards, you don't
N:   have to re-upload the package just to adjust the Standards-Version control
N:   field. However, please remember to update this field next time you upload
N:   the package.
N:   
N:   See /usr/share/doc/debian-policy/upgrading-checklist.txt.gz in the
N:   debian-policy package for a summary of changes in newer versions of
N:   Policy.
N: 
N:   Please refer to
N:   https://www.debian.org/doc/debian-policy/upgrading-checklist.html for
N:   details.
N: 
N:   Visibility: info
N:   Show-Always: no
N:   Check: fields/standards-version
N: 
N:
I: pwru source: patch-not-forwarded-upstream [debian/patches/use-external-libpcap]
N: 
N:   According to the DEP-3 headers, this patch has not been forwarded
N:   upstream.
N:   
N:   Please forward the patch and try to have it included in upstream's version
N:   control system. If the patch is not suitable for that, please mention
N:   not-needed in the Forwarded field of the patch header.
N: 
N:   Please refer to social contract item 2, Coordination with upstream
N:   developers (Section 3.1.4) in the Debian Developer's Reference, Changes to
N:   the upstream sources (Section 4.3) in the Debian Policy Manual, and
N:   Bug#755153 for details.
N: 
N:   Visibility: info
N:   Show-Always: no
N:   Check: debian/patches/dep3
N:   Renamed from: send-patch
N: 
N:
I: pwru: spelling-error-in-binary prefered preferred [usr/bin/pwru]
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
N: 
N:
P: pwru source: trailing-whitespace [debian/control:42]
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
N: 

E: Lintian run failed (runtime error)

3. Licenses [4]: Good

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]: N/A

Summary,

Looking good, but a few lintian issues that needs to be looked at.

Please try the below on your packages...

[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.debian.org/sbuild

[3] lintian:

  * Command: lintian --display-info --verbose --fail-on error --info --pedantic --show-overrides (*.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

  Note: Please report false positives as bug reports against '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
  
Regards

Phil
    Needs work Phil Wyett at Jan. 5, 2025, 8:46 a.m. 
  2. doesnt build for me:

	cd _build && go generate -v github.com/cilium/pwru github.com/cilium/pwru/internal/byteorder github.com/cilium/pwru/internal/libpcap github.com/cilium/pwru/internal/pwru
src/github.com/cilium/pwru/build.go
Generating for aarch64
Supported targets:
	bpf
	bpfel
	bpfeb
	386
	amd64
	arm
	arm64
	arm64be
	armbe
	ppc64
	ppc64le
	s390
	s390x
	sparc
	sparc64

Error: "aarch64": unsupported target
exit status 1
    Needs work zeha@d.o at Jan. 8, 2025, 9:53 p.m. 
  3. Thanks for that report Chris. I've uploaded a new pwru package that now should build on ARM and PPC architectures.
    Loren M. Lang at Jan. 10, 2025, 12:39 p.m. 
  4. Review of this continues at https://salsa.debian.org/go-team/packages/pwru/-/merge_requests/

This mentors upload can be removed.
    Ready Otto Kekäläinen at April 11, 2025, 6:35 p.m.