Re: [PATCH 2/2] Add jq

[Jelle Licht]

2016-03-27 0:26 GMT+01:00 Leo Famulari <address@hidden>:

On Sat, Mar 26, 2016 at 11:53:24PM +0100, 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?

In this case I think the description should be changed.

>
> --
> 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?
> >

^ changed

0002-gnu-Add-jq.patch
Description: Text Data