From: Christopher Baines <mail@cbaines.net>
To: Nicolas Graves <ngraves@ngraves.fr>
Cc: 58381@debbugs.gnu.org
Subject: [bug#58381] [PATCH 1/3] gnu: Add go-github-com-go-piv-piv-go.
Date: Thu, 13 Oct 2022 11:58:31 +0100 [thread overview]
Message-ID: <875ygo7zqw.fsf@cbaines.net> (raw)
In-Reply-To: <878rlmwo2x.fsf@ngraves.fr>
[-- Attachment #1: Type: text/plain, Size: 2874 bytes --]
Nicolas Graves <ngraves@ngraves.fr> writes:
> On 2022-10-11 11:48, Christopher Baines wrote:
>
>> What relation does pcsc-lite have to this package, and why does it need
>> to be propagated?
>>
>> If this can be avoid, that would be good. Otherwise it would be good to
>> mention why the propagation is needed in a comment.
>
> For piv-go, in their README:
>
> To build on Linux, piv-go requires PCSC lite. To install on Debian-based distros, run:
> sudo apt-get install libpcsclite-dev
>
> ==> So probably no need to propagate here.
>
> ----
>
> For yubikey-agent:
>
> The package relies on pcscd (daemon) to run on linux, which is already
> available in guix. Maybe it's better to say that in the README ? Instead
> of the package, what it needs is actually the service.
>
> I've also written some tested lines for a yubikey-agent
> configuration/feature (as done in the RDE project) shepherd service,
> although as it is really tiny, I didn't know if I should've contributed
> this.
>
> It looks like that, and did work (it also depends on the acceptance of
> the patch https://issues.guix.gnu.org/52900 in this case):
>
> (define* (feature-yubikey-agent
> #:key
> (yubikey-agent yubikey-agent))
> "Configure Yubikey-Agent."
> (ensure-pred file-like? yubikey-agent)
>
> (define (get-system-services _)
> (list
> (service pcscd-service-type)
> (udev-rules-service
> 'yubikey
> (file->udev-rule
> "70-u2f.rules"
> (file-append libfido2 "/udev/rules.d/70-u2f.rules")))))
>
> ;; (define (get-home-services config)
> ;; (list
> ;; ;; MAYBE: It should be installed system-wide?
> ;; (simple-service
> ;; 'yubikey-agent-add-yubikey-agent-package
> ;; home-profile-service-type
> ;; (list yubikey-agent))
> ;; (simple-service
> ;; 'start-yubikey-agent-at-startup
> ;; home-shepherd-service-type
> ;; (list (shepherd-service
> ;; (documentation "Run the yubikey-agent.")
> ;; (provision '(yubikey-agent))
> ;; (requirement '())
> ;; (start #~(make-forkexec-constructor
> ;; (list (string-append
> ;; #$yubikey-agent "/bin/yubikey-agent")
> ;; "-l" "/tmp/yubikey-agent.sock")))
> ;; (stop #~(make-kill-destructor)))))))
>
> (feature
> (name 'yubikey-agent)
> (values `((yubikey-agent . ,yubikey-agent)))
> (system-services-getter get-system-services)
> ;; (home-services-getter get-home-services)
> ))
>
>
> I'm not sure what would be the guideline here regarding doing or
> avoiding propagations. WDYT? (I'm sending a V2 once I have a clearer
> idea of what makes sense).
I'd try avoiding all the propagated inputs and then tackle the issue of
getting the service working separately.
Thanks,
Chris
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]
next prev parent reply other threads:[~2022-10-13 11:21 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-08 16:36 [bug#58381] yubikey-agent Nicolas Graves via Guix-patches via
2022-10-08 16:39 ` [bug#58381] [PATCH 1/3] gnu: Add go-github-com-go-piv-piv-go Nicolas Graves via Guix-patches via
2022-10-08 16:39 ` [bug#58381] [PATCH 2/3] gnu: Add go-github-com-twpayne-go-pinentry-minimal Nicolas Graves via Guix-patches via
2022-10-08 16:39 ` [bug#58381] [PATCH 3/3] gnu: Add yubikey-agent Nicolas Graves via Guix-patches via
2022-10-11 10:50 ` Christopher Baines
2022-10-11 10:48 ` [bug#58381] [PATCH 1/3] gnu: Add go-github-com-go-piv-piv-go Christopher Baines
2022-10-11 12:18 ` Nicolas Graves via Guix-patches via
2022-10-13 10:58 ` Christopher Baines [this message]
2022-10-23 9:03 ` Nicolas Graves via Guix-patches via
2022-10-23 9:03 ` [bug#58381] [PATCH 2/3] gnu: Add go-github-com-twpayne-go-pinentry-minimal Nicolas Graves via Guix-patches via
2022-10-23 9:03 ` [bug#58381] [PATCH 3/3] gnu: Add yubikey-agent Nicolas Graves via Guix-patches via
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=875ygo7zqw.fsf@cbaines.net \
--to=mail@cbaines.net \
--cc=58381@debbugs.gnu.org \
--cc=ngraves@ngraves.fr \
/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 external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.