From: Skyler Ferris via Guix-patches via <guix-patches@gnu.org>
To: 69131@debbugs.gnu.org
Subject: [bug#69131] [PATCH 1/2] gnu: Add passt.
Date: Fri, 16 Feb 2024 03:02:43 +0000 [thread overview]
Message-ID: <44375893-d991-49cf-a050-68ce493de299@protonmail.com> (raw)
In-Reply-To: <cf279ef242bd28abcb8c73db99f0b9f57ddf99c8.1707938222.git.~@wolfsden.cz>
"Add passt" review notes:
The following checklist is based on the items listed in the manual (22.7
"Submitting Patches"). Each checklist item may have one of the following
statuses:
C: Commitable. This item looks as expected for a final commit.
CC: Commitable with Context. Like commitable, but some notes have been
added providing context about anything notable or unusual. These notes
are provided for transparency or to set expectations for the committer
and should not be interpreted as meaning that I object to the patch.
MR: Minor Repair. There is something about the patch which I think
should ideally be changed, but I do not think that the commit should be
held up for this reason alone.
A: Attention. Something in the patch needs attention from the author,
the committer, or both. Resolving the matter may or may not require a
change to the patch.
Blank: I did not review this aspect of the patch.
[CC] Applies cleanly (commit bd87416648929f38c0173f047776d7675ea8a10d)
[CC] Cryptographic hash
[C ] Synopsis & Description
[C ] Package license matches source license
[C ] No compiler warnings
[CC] `guix lint` output
[MR] `guix style` output
[C ] Package builds (my host is an x86_64 Xen guest)
[ ] Builds for other architectures
[C ] No bundled software
[ ] Works in container with minimal inputs
[C ] `guix size` output
[C ] Dependent packages (`guix refresh --list-dependent PACKAGE`)
[C ] `guix build --rounds=2`
[C ] Gender-neutral wording
[C ] One set of related changes
[ ] Simulated guix pull
# Patch Application
The copyright statement in the first patch does not apply cleanly
because commit 6002e9e7711136373de5a6325769e9c7d11032c9 recently added
it already, but git successfully processed that diff after manually
deleting the chunk from the patch.
# Cryptographic Signatures
I was not able to locate signatures for either the source tarball or the
git commits.
# Guix Lint Output
The linter printed two notes, one that the 'generic-html' updater failed
to find upstream releases and another indicated that archives are not
available from either Software Heritage or Disarchive. As I understand
it, these issues issues should not block the package. Not every package
has an automatic updater (though it would be nice if they did) and the
release page for this project is atypical. The 'archival' section in the
"Invoking 'guix lint'" chapter (9.8) indicates that the Software
Heritage will pick up the source at some point in the future.
# Guix Style output (Minor repair)
The tool made some reasonable-looking changes. I'm not sure if it would
be appropriate for me to add a new patch to this thread that applies
these changes.
# Runs in a container with minimal inputs
This package wants to use QEMU, and my machine is already a Xen guest. I
know that there are some complications when it comes to nested
virtualization. The demo script (at #$output/share/doc/passt/demo.sh)
did not "just work" for me, but it's not clear to me that this is a
problem with the package. I tried running it with the following:
```
$ # Extra inputs are for the shell script
$ ./pre-inst-env guix shell -C iproute2 coreutils util-linux-with-udev
passd
[env]$
/gnu/store/wgzlv2chrxmk4y4m4fxx8g7x0z59cb98-passt-2023_12_30.f091893/share/doc/passt/demo.sh
```
next prev parent reply other threads:[~2024-02-16 3:03 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-14 19:17 [bug#69131] [PATCH 1/2] gnu: Add passt Tomas Volf
2024-02-14 19:19 ` [bug#69131] [PATCH 2/2] gnu: podman: Update to 4.9.3 Tomas Volf
2024-02-16 3:01 ` Skyler Ferris via Guix-patches via
2024-02-16 21:16 ` Tomas Volf
2024-02-16 3:03 ` Skyler Ferris via Guix-patches via
2024-02-16 3:02 ` Skyler Ferris via Guix-patches via [this message]
2024-02-16 21:19 ` [bug#69131] [PATCH 1/2] gnu: Add passt Tomas Volf
2024-02-16 21:14 ` [bug#69131] [PATCH v2 " Tomas Volf
2024-02-16 21:14 ` [bug#69131] [PATCH v2 2/2] gnu: podman: Update to 4.9.3 Tomas Volf
2024-03-02 16:25 ` bug#69131: [PATCH v2 1/2] gnu: Add passt Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=44375893-d991-49cf-a050-68ce493de299@protonmail.com \
--to=guix-patches@gnu.org \
--cc=69131@debbugs.gnu.org \
--cc=skyvine@protonmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).