Re: [PATCH] Add msgpack

[Leo Famulari]

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?