unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Brice Waegeneire <brice@waegenei.re>
To: mail@cbaines.net
Cc: 40631@debbugs.gnu.org
Subject: [bug#40631] [PATCH 0/4] Build QEMU with smartcard support, to get GNOME Boxes working
Date: Tue, 28 Apr 2020 09:07:59 +0000	[thread overview]
Message-ID: <0d9b8d67ec45e445eac9b5ecbae268ad@waegenei.re> (raw)
In-Reply-To: <877dyh7tcn.fsf@cbaines.net>

Hello Christopher,

I was thinking of packaging libcacard the other day, thankfully you've 
already
done it.

If I understand correctly softhsm is needed to test libcacard, but it's 
tests
can't be enabled because our gnutls is missing some features and 
rebuilding it
would rebuild the world, right?

> +    (name "softhsm")
> +    (version "2.5.0")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                    "https://dist.opendnssec.org/source/"
> +                    "softhsm-" version ".tar.gz"))

The latest version is 2.6.0, even though the home-page don't list it, 
it's present
in the source you defined or in their Github page[0].


> +         (add-after 'unpack 'patch
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (substitute* "tests/setup-softhsm2.sh"
> +               (("\\/usr\\/lib64\\/pkcs11\\/libsofthsm2\\.so")
> +                (string-append (assoc-ref inputs "softhsm")
> +                               "/lib/softhsm/libsofthsm2.so")))
> +             #t)))))

Instead of 'patch' 'patch-tests' would be more descriptive.

LGTM otherwise, building and linting is successful.

[0]: https://github.com/opendnssec/SoftHSMv2/releases

- Brice

  parent reply	other threads:[~2020-04-28  9:30 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-14 19:36 [bug#40631] [PATCH 0/4] Build QEMU with smartcard support, to get GNOME Boxes working Christopher Baines
2020-04-14 19:44 ` [bug#40631] [PATCH 1/4] gnu: Add softhsm Christopher Baines
2020-04-14 19:44   ` [bug#40631] [PATCH 2/4] gnu: Add libcacard Christopher Baines
2020-04-14 19:44   ` [bug#40631] [PATCH 3/4] gnu: qemu: Add libcacard as an input to enable smartcard support Christopher Baines
2020-04-14 19:44   ` [bug#40631] [PATCH 4/4] gnu: spice: " Christopher Baines
2020-04-28  9:07 ` Brice Waegeneire [this message]
2020-04-28 10:11   ` [bug#40631] [PATCH 0/4] Build QEMU with smartcard support, to get GNOME Boxes working Brice Waegeneire
2020-05-03  9:45     ` Christopher Baines
2020-05-03  9:43   ` bug#40631: " Christopher Baines

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=0d9b8d67ec45e445eac9b5ecbae268ad@waegenei.re \
    --to=brice@waegenei.re \
    --cc=40631@debbugs.gnu.org \
    --cc=mail@cbaines.net \
    /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).