From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59779) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dcfyN-0006cI-NW for guix-patches@gnu.org; Tue, 01 Aug 2017 18:49:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dcfyK-0003rC-Hl for guix-patches@gnu.org; Tue, 01 Aug 2017 18:49:07 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:34757) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dcfyK-0003qs-Dv for guix-patches@gnu.org; Tue, 01 Aug 2017 18:49:04 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dcfyK-00076x-51 for guix-patches@gnu.org; Tue, 01 Aug 2017 18:49:04 -0400 Subject: bug#27907: [PATCH] graph: Provide access to the package record in the emit Resent-To: guix-patches@gnu.org Resent-Message-ID: References: <87lgn3w0n6.fsf@gnu.org> From: Roel Janssen Message-ID: <87vam6q50t.fsf@gnu.org> In-reply-to: <87lgn3w0n6.fsf@gnu.org> Date: Wed, 02 Aug 2017 00:47:53 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Ludovic =?UTF-8?Q?Court=C3=A8s?= Cc: 27907-done@debbugs.gnu.org Ludovic Courtès writes: > Roel Janssen 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