From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH] Add msgpack Date: Mon, 13 Jun 2016 12:58:52 -0400 Message-ID: <20160613165852.GA31846@jasmine> References: <87wpmbuv6m.fsf@openmailbox.org> <20160606140737.GA26555@jasmine> <20160611030655.GA24837@jasmine> <87lh2b7009.fsf@openmailbox.org> <20160612001750.GA9854@jasmine> <87eg836nm3.fsf@openmailbox.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:35648) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bCVCf-0003xD-58 for guix-devel@gnu.org; Mon, 13 Jun 2016 12:59:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bCVCa-0008Bs-UH for guix-devel@gnu.org; Mon, 13 Jun 2016 12:59:09 -0400 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:40459) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bCVCY-000892-L3 for guix-devel@gnu.org; Mon, 13 Jun 2016 12:59:04 -0400 Content-Disposition: inline In-Reply-To: <87eg836nm3.fsf@openmailbox.org> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Lukas Gradl Cc: guix-devel@gnu.org On Sat, Jun 11, 2016 at 11:24:23PM -0500, Lukas Gradl wrote: > Oh, that is true. I searched around, but I could not find anything > about zbuffer.h[pp]. I am not sure what it is used for. FWIW, it is > only referenced in src/Makefile, src/Makefile.in, src/Makefile.am > CMakeLists.txt and in the tests: [...] > Judging from this, I am inclined to think that msgpack does not use > these two headers for anything but tests. But I am not sure about > this. I think that another application would call the zbuffer functions, although the string 'zbuffer' does not exist in the opendht source code. But, we should make sure msgpack will work for other calling applications that might be added later. On #guix, bavier suggested we patch 'msgpack.pc.in'. Specifically, we should append the include flag to 'Cflags:' and create a 'Libs.private:' field for the -L and -l flags. There are some examples in 'gnu/packages'. More details on the IRC log: https://gnunet.org/bot/log/guix/2016-06-13#T1056507 I've cc-ed Eric since I don't know much about pkg-config. > FWIW, I looked at what other distributions do: > > The Arch PKGBUILD does not refer to Zlib at all: > https://git.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?h=packages/msgpack-c > > Gentoo has Zlib for tests: > https://gitweb.gentoo.org/repo/gentoo.git/tree/dev-libs/msgpack/msgpack-1.1.0.ebuild > > Debian seems to not refer to Zlib: > https://packages.debian.org/jessie/libmsgpackc2 > https://packages.debian.org/jessie/libmsgpack3 > https://packages.debian.org/jessie/libmsgpack-dev I think that zlib is so ubiquitous that it's a safe assumption it will can be found in the environment on the '/usr' distros. > This Nix expression does not mention zlib: > https://github.com/NixOS/nixpkgs/blob/f199be0fafcff0713dd4340b4373e575294aad58/pkgs/development/libraries/libmsgpack/generic.nix#L41 An oversight? I don't know the Nix equivalent of `guix refresh -l` to check which of their packages use msgpack.