Re: [PATCH] gnu: Add libdispatch.

[ng0]

Leo Famulari <address@hidden> 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