Re: [PATCH] gnu: Add wiredtiger.

[宋文武]

address@hidden (Ludovic Courtès) writes:

> 宋文武 <address@hidden> skribis:
>
>> * gnu/packages/database.scm (wiredtiger): New variable.
>
> [...]
>
>> +    (arguments
>> +     '(#:tests? #f ; XXX: timeout
>
> Could you elaborate a bit?  :-)
OK, and it turns out that skip 1 test is enough.

>
>> +    (license gpl3) ; or GPL-2
>
> v3-only or v2-only, right?
>
Yes, sources say “See the file LICENSE” and there is no ‘later’
in the LICENSE file.

>> +    ;; WiredTiger requires a 64-bit build.
>> +    (supported-systems '("x86_64-linux" "mips64el-linux"))))
>
> If there’s a file or something that explains this, could you mention it
> in the comment?
Yep, ‘configure.ac’ check it.


Updated:

0001-gnu-Add-wiredtiger.patch
Description: Text Data

Thanks for the review!