From mboxrd@z Thu Jan 1 00:00:00 1970 From: ng0 Subject: Re: [PATCH] gnu: Add libdispatch. Date: Fri, 28 Oct 2016 11:23:58 +0000 Message-ID: <87bmy4lnsh.fsf@we.make.ritual.n0.is> References: <87poo4m84e.fsf@we.make.ritual.n0.is> <20161002003905.GA24418@jasmine> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:47479) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c05Gb-0001Ws-57 for guix-devel@gnu.org; Fri, 28 Oct 2016 07:24:14 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c05GW-0006BW-MX for guix-devel@gnu.org; Fri, 28 Oct 2016 07:24:09 -0400 Received: from aibo.runbox.com ([91.220.196.211]:55308) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1c05GW-0006BG-G0 for guix-devel@gnu.org; Fri, 28 Oct 2016 07:24:04 -0400 In-Reply-To: <20161002003905.GA24418@jasmine> (Leo Famulari's message of "Sat, 1 Oct 2016 20:39:05 -0400") 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: Leo Famulari Cc: guix-devel@gnu.org Leo Famulari writes: > On Fri, Sep 16, 2016 at 10:50:09AM +0000, ng0 wrote: >> Subject: [PATCH 1/4] gnu: Add libkqueue. >> >> * gnu/local.mk: Include gnu/packages/libdispatch.scm . >> * gnu/packages/libdispatch.scm: Add new file. >> * gnu/packages/libdispatch.scm (libkqueue): New variable. > > LGTM > >> Subject: [PATCH 2/4] gnu: Add libpthread_workqueue. >> >> * gnu/packages/libdispatch.scm (libpthread_workqueue): New variable. > >> + (version "0.9.1") > > 0.9.2 is available upstream: > https://github.com/mheily/libpwq/releases > > Can you try using that? Ok >> Subject: [PATCH 3/4] gnu: Add libblocks-runtime. >> >> * gnu/packages/libdispatch.scm (libblocks-runtime): New variable. > >> + (add-after 'unpack 'autogen >> + (lambda _ >> + (zero? (system* "autoreconf" "--install" "--symlink" "--verbose"))))))) > > Why use --symlink? I have no idea why I did this. I'll remove it in the new version of the patches. >> Subject: [PATCH 4/4] gnu: Add libdispatch. >> >> * gnu/packages/libdispatch.scm (libdispatch): New variable. > >> + (arguments >> + `(#:tests? #f)) > > The tests can be run with `ctest` [0]. > > The test suite can be built by setting "-DDISPATCH_ENABLE_TEST_SUITE=1" > in #:configure-flags. However, when I tried that, the build log included > "No compiler support for Blocks; test suite will not be built.", and the > tests were indeed not built. > > Does that mean that libblocks-runtime is not working? An ebuild for this package contains this: src_install() { multilib_foreach_abi ustr_install install-multilib-linux #TODO this is redundant and stupid, why does libdispatch need this file cp "${S}/BlocksRuntime/Block_private.h" "${ED}/usr/include" || die # dodoc ChangeLog README README-DEVELOPERS AUTHORS NEWS TODO } ( source: https://data.gpo.zugaina.org/netcrave/dev-libs/libblocksruntime/libblocksruntime-9999.ebuild ) Maybe Blocks_private.h is expected in that location but not copied by upstream. I will take a look once the test suite runs. This is libblocksruntime, not libdispatch but I think I packaged that as dependency for libdispatch. > Also, the libdispatch tarball bundles Click [0] at 'thirdparty/click'. We > should remove that and use our Click package. Ok > [0] > https://github.com/nickhutchinson/libdispatch#testing-with-ctest > > [1] > http://click.pocoo.org/5/ > -- ♥Ⓐ ng0 | ng0.chaosnet.org