unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Marius Bakke <mbakke@fastmail.com>
To: Tobias Geerinckx-Rice <me@tobias.gr>, 26256@debbugs.gnu.org
Subject: bug#26256: [PATCH 5/6] gnu: Add userspace-rcu.
Date: Mon, 27 Mar 2017 22:08:25 +0200	[thread overview]
Message-ID: <8737dywkhy.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me> (raw)
In-Reply-To: <cdd6892d-5182-ee07-7452-8df83136e3d0@tobias.gr>

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

Tobias Geerinckx-Rice <me@tobias.gr> writes:

> Marius,
>
> On 25/03/17 21:30, Marius Bakke wrote:
>> * gnu/packages/linux.scm (userspace-rcu): New variable.
>
> This already exists as liburcu in datastructures.scm (I needed it for
> the Knot DNS server), with two main differences:

Aaaah.. how did I miss that, thanks!

>> +    (native-inputs
>> +     `(("perl-test-harness" ,perl-test-harness))) ; for the 'prove' command
>
> Building liburcu with ‘perl’ or ‘perl-test-harness’ gives the same
> result (Files=3, Tests=78). I know nothing about Perl. Is there a reason
> to prefer ‘perl-test-harness’? It's about .5 MiB heavier.

I don't know either, I just searched around for the `prove` command and
found it was in "perl-test-harness". Apparently it's in "perl" too!

>> +    (license
>> +     ;; This library is distributed under LGPL2.1+, but includes some files
>> +     ;; covered by other licenses. The LICENSE file has full details.
>> +     (list license:lgpl2.1+
>> +           license:gpl3+                         ; most tests are gpl2+; tap.sh is gpl3+
>> +           license:bsd-2                         ; tests/utils/tap/tap.[ch]
>> +           license:expat                         ; urcu/uatomic/*
>> +           ;; A few files use different variants of the MIT/X11 license.
>> +           (license:x11-style "file://LICENSE"
>> +                              "See LICENSE in the distribution for details.")))))
>> +
>
> Wow. As usual you're a lot better at sniffing out licences than I am.

Ha, but at least you got it right (per the other discussion on this
thread) ;-)

> I also prefer your ‘synopsis’ to mine.

I can update the synopsis of the existing package, and add @code{}
wrappers to your arguably better description; as well as adding comments
about the other licenses. How does that sound? :)

PS: The PGP signature of this email is bad. If you are in fact Tobias,
please reply with a signed email stating "I will not use important
software from my host distro any longer" :-P

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

  parent reply	other threads:[~2017-03-27 20:09 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20170325202717.14345-1-mbakke@fastmail.com>
     [not found] ` <20170325203017.14931-1-mbakke@fastmail.com>
2017-03-27 11:38   ` bug#26256: [PATCH 1/6] Revert "gnu: snappy: Update to 1.1.4." Ludovic Courtès
     [not found]   ` <20170325203017.14931-2-mbakke@fastmail.com>
2017-03-27 11:39     ` bug#26256: [PATCH 2/6] gnu: xmlstarlet: Disable failing tests Ludovic Courtès
     [not found]   ` <20170325203017.14931-3-mbakke@fastmail.com>
     [not found]     ` <87wpbdw04q.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me>
2017-03-27 11:48       ` bug#26256: [PATCH 3/6] gnu: Add ceph Ludovic Courtès
2017-03-27 20:13         ` Marius Bakke
2017-03-28  8:57           ` Marius Bakke
2017-03-28  9:43           ` Ludovic Courtès
     [not found]   ` <20170325203017.14931-4-mbakke@fastmail.com>
2017-03-27 11:50     ` bug#26256: [PATCH 4/6] gnu: fio: Enable rbd support Ludovic Courtès
2017-03-27 22:10       ` Marius Bakke
2017-03-28  9:45         ` Ludovic Courtès
     [not found]   ` <20170325203017.14931-5-mbakke@fastmail.com>
2017-03-27 11:51     ` bug#26256: [PATCH 5/6] gnu: Add userspace-rcu Ludovic Courtès
2017-03-27 19:13       ` Marius Bakke
2017-03-27 22:57         ` Marius Bakke
2017-03-27 23:41           ` Marius Bakke
2017-03-28  8:17             ` Ludovic Courtès
2017-03-28  8:14         ` Ludovic Courtès
2017-03-27 19:49     ` Tobias Geerinckx-Rice
2017-03-27 19:54       ` Tobias Geerinckx-Rice
2017-03-27 20:08       ` Marius Bakke [this message]
2017-03-27 22:21         ` Tobias Geerinckx-Rice
2017-03-27 22:31           ` Marius Bakke
2017-03-27 22:59             ` Tobias Geerinckx-Rice
2017-03-27 23:05               ` Marius Bakke
2017-03-28 11:16             ` Ludovic Courtès
     [not found]   ` <20170325203017.14931-6-mbakke@fastmail.com>
2017-03-27 11:52     ` bug#26256: [PATCH 6/6] gnu: Add multipath-tools Ludovic Courtès
2017-03-28 15:10 ` bug#26256: [PATCH 0/6] Add ceph + multipath-tools Marius Bakke

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=8737dywkhy.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me \
    --to=mbakke@fastmail.com \
    --cc=26256@debbugs.gnu.org \
    --cc=me@tobias.gr \
    /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).