[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] Add jq
From: |
Alex Griffin |
Subject: |
Re: [PATCH 2/2] Add jq |
Date: |
Sat, 26 Mar 2016 18:15:53 -0500 |
Usually a second package is added which is just a variant on the first.
Something like this:
(define-public jq-no-oniq
(package
(inherit jq)
(name "jq-no-onig")
(inputs '())))
But in this case I'm not sure it's worth it. Maybe just remove the part
about zero dependencies from the description instead.
--
Alex Griffin
On Sat, Mar 26, 2016, at 05:53 PM, Jelle Licht wrote:
> From jq's README:
>
>> To get
regexp support you'll also need to install Oniguruma (note that jq's
tests require regexp support to pass).
>
> So it seems to be an optional dependency (which allows tests to pass and adds
> functionality). I could make it a build-time dependency, or not.
> What is the 'guix' way of handling these cases?
>
> --
> Jelle
>
>
> 2016-03-26 23:45 GMT+01:00 Leo Famulari <address@hidden>:
>> On Sat, Mar 26, 2016 at 11:06:15PM +0100, Jelle Licht wrote:
>>
> + (inputs
>>
> + `(("oniguruma" ,oniguruma)))
>>
>>
>>
[...]
>>
>>
>>
> + (description "jq is like sed for JSON data – you can use it to slice and
>>
> +filter and map and transform structured data with the same ease that sed,
> awk,
>>
> +grep and friends let you play with text. It is written in portable C, and it
>>
> +has zero runtime dependencies. jq can mangle the data format that you have
>>
^-----------------------^
>>
>>
>>
Since oniguruma is listed as a run-time dependency, does this mean the
>>
description is wrong? Or is oniguruma only needed to build?