[debbugs-tracker] bug#27907: closed ([PATCH] graph: Provide access to the package record in the emit)

[GNU bug Tracking System]

Your message dated Wed, 02 Aug 2017 00:47:53 +0200
with message-id <address@hidden>
and subject line Re: [bug#27907] [PATCH] graph: Provide access to the package 
record in the emit
has caused the debbugs.gnu.org bug report #27907,
regarding [PATCH] graph:  Provide access to the package record in the emit
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
27907: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=27907
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: [PATCH] graph: Provide access to the package record in the emit Date: Tue, 01 Aug 2017 16:40:05 +0200 User-agent: mu4e 0.9.18; emacs 25.2.1

0001-graph-Provide-access-to-the-package-record-in-the-em.patch
Description: Text Data

Dear Guix,

I would like to expand the Cypher back-end and in the long run add a
SPARQL graph back-end to GNU Guix.  For this, I will need to have access
to the package records inside the emit-* functions.

This patch makes this change by essentially changing the "label"
parameter of the emit-* functions passed as "(node-label head)" into a
"node" parameter, passed as "head".

The rest of the patch adapts the current emit-* functions to this
change.

I tested the Graphviz, D3js, and Cypher back-ends, and all seem to work
as before.

Is it OK to apply this change?

Kind regards,
Roel Janssen

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: [bug#27907] [PATCH] graph: Provide access to the package record in the emit
>
>
>
>
> Date: 
>
> Wed, 02 Aug 2017 00:47:53 +0200
>
>
>
>
> User-agent: 
>
> mu4e 0.9.18; emacs 25.1.1
>
>
>
>
> Ludovic Courtès writes:
>
> > Roel Janssen <address@hidden> skribis:
> >
> >> I would like to expand the Cypher back-end and in the long run add a
> >> SPARQL graph back-end to GNU Guix.  For this, I will need to have access
> >> to the package records inside the emit-* functions.
> >>
> >> This patch makes this change by essentially changing the "label"
> >> parameter of the emit-* functions passed as "(node-label head)" into a
> >> "node" parameter, passed as "head".
> >>
> >> The rest of the patch adapts the current emit-* functions to this
> >> change.
> >>
> >> I tested the Graphviz, D3js, and Cypher back-ends, and all seem to work
> >> as before.
> >>
> >> Is it OK to apply this change?
> >
> > Sure, looks good to me!
> >
> > Ludo’.
>
> Thanks! I pushed this patch.
>
> Kind regards,
> Roel Janssen
>
>
> --- End Message ---