unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Leo Famulari <leo@famulari.name>
To: Lukas Gradl <lgradl@openmailbox.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] Add msgpack
Date: Sat, 11 Jun 2016 20:17:50 -0400	[thread overview]
Message-ID: <20160612001750.GA9854@jasmine> (raw)
In-Reply-To: <87lh2b7009.fsf@openmailbox.org>

On Sat, Jun 11, 2016 at 06:56:38PM -0500, Lukas Gradl wrote:
> It seems that zlib is only required for tests.  In README and README.md
> zlib is never  mentioned among the dependencies, which also supports
> that it is only needed for tests.  I was not aware of that before (Shame
> on me).

Is zlib really only required for tests? I see this:

$ grep -rIi zlib /gnu/store/xn8z7k6j7zm4qz14bm29fgk0kwwvz3c4-msgpack-1.4.1
/gnu/store/xn8z7k6j7zm4qz14bm29fgk0kwwvz3c4-msgpack-1.4.1/include/msgpack/zbuffer.h:#include <zlib.h>
/gnu/store/xn8z7k6j7zm4qz14bm29fgk0kwwvz3c4-msgpack-1.4.1/include/msgpack/zbuffer.hpp:#include <zlib.h>

So, there are headers in the msgpack output that require zlib.

If you are at the stage where you are using Ring based on these
packages, I wonder if it is not using the part of msgpack that uses
zlib? Or, if it's finding zlib in the environment (if you are on a
foreign distro)?

Or have I misunderstood?

  reply	other threads:[~2016-06-12  0:18 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-30 20:50 [PATCH] Add msgpack Lukas Gradl
2016-05-31 21:45 ` Leo Famulari
2016-06-01  4:49 ` Efraim Flashner
2016-06-03  0:50   ` Lukas Gradl
2016-06-06 14:07 ` Leo Famulari
2016-06-11  3:06   ` Leo Famulari
2016-06-11 23:56     ` Lukas Gradl
2016-06-12  0:17       ` Leo Famulari [this message]
2016-06-12  4:24         ` Lukas Gradl
2016-06-13 16:58           ` Leo Famulari
2016-06-13 17:59             ` Leo Famulari
2016-06-17 15:13               ` Lukas Gradl
2016-06-19  3:44                 ` Lukas Gradl
2016-06-20 17:09                   ` Leo Famulari
2016-06-21 13:55                     ` Lukas Gradl
2016-06-21 16:06                       ` Leo Famulari
2016-06-21 15:59                     ` Lukas Gradl
2016-06-21 16:49                       ` Efraim Flashner
2016-06-22  6:05                         ` Lukas Gradl
2016-06-22  5:56                       ` Lukas Gradl
2016-06-21  8:38                   ` Ludovic Courtès
2016-06-21 13:31                     ` Lukas Gradl
2016-06-21 14:11                       ` Ludovic Courtès
2016-06-25 15:31                 ` Leo Famulari
2016-06-25 16:40                   ` Lukas Gradl

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=20160612001750.GA9854@jasmine \
    --to=leo@famulari.name \
    --cc=guix-devel@gnu.org \
    --cc=lgradl@openmailbox.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 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).