all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Marius Bakke <mbakke@fastmail.com>
To: Mike Gerwitz <mtg@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 0/2] gnu: Add libpcsclite
Date: Thu, 27 Oct 2016 10:03:43 +0100	[thread overview]
Message-ID: <877f8u17v4.fsf@duckhunt.i-did-not-set--mail-host-address--so-tickle-me> (raw)
In-Reply-To: <8737jiha1y.fsf@gnu.org>

[-- Attachment #1: Type: text/plain, Size: 1299 bytes --]

Mike Gerwitz <mtg@gnu.org> writes:

> Marius:
>
> Thanks for your mentoring on this. :)

You're welcome! :)

>
> On Mon, Oct 24, 2016 at 17:21:18 +0100, Marius Bakke wrote:
>> I'll continue working on getting ccid integrated and eventually make a
>> pcscd service for GuixSD.
>
> Is there anything you'd like help on?  I'd be happy to test whatever you
> come up with as soon as it's available.

Not right now, but thanks. The difficulty with getting pcscd working on
GuixSD is that the daemon expects to find all drivers in one location
(e.g. /var/lib/pcsc/drivers). So we need to populate this directory with
drivers before starting the service.

On foreign (non-GuixSD) distros this is not a problem, as gpg users are
expected to use the pcscd service provided by their system anyway.

Anyway, the patches look good, I've made the following minor changes:

* moved pcsc-lite to the newly created "security-token.scm";
* made eudev a regular input instead of propagated;
* renamed the gnupg patch phase and removed superfluous "outputs" key;
* added a configure flag to pcsc so that GuixSD users can manually add
  drivers to "/var/lib/pcsc/drivers" (default is $out/lib/pcsc/drivers).

Also adjusted the commit messages accordingly. Pushed as
cd9ed6c1cbd9fdcfa38825b8109a646fb968b7a8! Thank you!

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 454 bytes --]

      reply	other threads:[~2016-10-27  9:03 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-22 16:24 [PATCH 0/2] gnu: Add libpcsclite Mike Gerwitz
2016-10-22 16:24 ` [PATCH 1/2] " Mike Gerwitz
2016-10-22 16:24 ` [PATCH 2/2] gnu: gnupg: libpcsclite propagated input Mike Gerwitz
2016-10-22 21:16 ` [PATCH 0/2] gnu: Add libpcsclite Marius Bakke
2016-10-23  1:30   ` Mike Gerwitz
2016-10-24 16:21     ` Marius Bakke
2016-10-22 16:04       ` [PATCH 1/2] gnu: Add pcsc-lite Mike Gerwitz
2016-10-22 16:06       ` [PATCH 2/2] gnu: gnupg: patch scdaemon libpcsclite path Mike Gerwitz
2016-10-24 18:03       ` [PATCH 0/2] gnu: Add libpcsclite Efraim Flashner
2016-10-27  1:09       ` Mike Gerwitz
2016-10-27  9:03         ` Marius Bakke [this message]

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=877f8u17v4.fsf@duckhunt.i-did-not-set--mail-host-address--so-tickle-me \
    --to=mbakke@fastmail.com \
    --cc=guix-devel@gnu.org \
    --cc=mtg@gnu.org \
    /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.