[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add msgpack
From: |
Leo Famulari |
Subject: |
Re: [PATCH] Add msgpack |
Date: |
Sat, 11 Jun 2016 20:17:50 -0400 |
User-agent: |
Mutt/1.6.0 (2016-04-01) |
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?
- Re: [PATCH] Add msgpack, Efraim Flashner, 2016/06/01
- Re: [PATCH] Add msgpack, Leo Famulari, 2016/06/06
- Re: [PATCH] Add msgpack, Leo Famulari, 2016/06/10
- Re: [PATCH] Add msgpack, Lukas Gradl, 2016/06/11
- Re: [PATCH] Add msgpack,
Leo Famulari <=
- Re: [PATCH] Add msgpack, Lukas Gradl, 2016/06/12
- Re: [PATCH] Add msgpack, Leo Famulari, 2016/06/13
- Re: [PATCH] Add msgpack, Leo Famulari, 2016/06/13
- Re: [PATCH] Add msgpack, Lukas Gradl, 2016/06/17
- Re: [PATCH] Add msgpack, Lukas Gradl, 2016/06/18
- Re: [PATCH] Add msgpack, Leo Famulari, 2016/06/20
- Re: [PATCH] Add msgpack, Lukas Gradl, 2016/06/21
- Re: [PATCH] Add msgpack, Leo Famulari, 2016/06/21
- Re: [PATCH] Add msgpack, Lukas Gradl, 2016/06/21
- Re: [PATCH] Add msgpack, Efraim Flashner, 2016/06/21