From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ludovic =?UTF-8?Q?Court=C3=A8s?= Subject: bug#34884: guix describe fails with --format=json and --format=recutils Date: Sun, 17 Mar 2019 16:40:27 +0100 Message-ID: <87ef75y0es.fsf@gnu.org> References: <871s37unyz.fsf@ambrevar.xyz> <87zhpuucxh.fsf@gmail.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" Return-path: Received: from eggs.gnu.org ([209.51.188.92]:42548) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1h5Y5B-00088R-CT for bug-guix@gnu.org; Sun, 17 Mar 2019 11:52:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1h5XuI-0000Kf-RO for bug-guix@gnu.org; Sun, 17 Mar 2019 11:41:03 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:34036) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1h5XuI-0000KI-LR for bug-guix@gnu.org; Sun, 17 Mar 2019 11:41:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1h5XuI-0003cv-F2 for bug-guix@gnu.org; Sun, 17 Mar 2019 11:41:02 -0400 Sender: "Debbugs-submit" Resent-Message-ID: In-Reply-To: <87zhpuucxh.fsf@gmail.com> (Oleg Pykhalov's message of "Sat, 16 Mar 2019 17:11:22 +0300") List-Id: Bug reports for GNU Guix List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-guix-bounces+gcggb-bug-guix=m.gmane.org@gnu.org Sender: "bug-Guix" To: Oleg Pykhalov Cc: Pierre Neidhardt , 34884@debbugs.gnu.org --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Hi Oleg, Oleg Pykhalov skribis: > JSON format: > > oleg@guixsd ~/src/guix$ ./pre-inst-env env GUIX_PACKAGE_PATH=3D$HOME/src/= guix-wigust:/tmp/noexist:/tmp/noexist:/tmp/noexist:/tmp/noexist:/tmp/noexis= t:/tmp/noexist:/tmp/noexist:/tmp/noexist:/tmp/noexist:/tmp/noexist:/tmp/noe= xist:/tmp/noexist:/tmp/noexist:/tmp/noexist:/tmp/noexist:/tmp/noexist:/tmp/= noexist:/tmp/noexist guix describe -p ~/.config/guix/current --format=3Djson > [{"name":"guix","url":"https://gitlab.wugi.info/guix/guix.git","commit":"= 4161deb4549c39b7d4801cc8aa63c365d19fc649"},{"name":"guix-wigust","url":"htt= ps://gitlab.wugi.info/guix/guix-wigust.git","commit":"f6dfa5fc08824ebe5bdc4= 2ea35ff0e040245c8c0"}] > {"name":"GUIX_PACKAGE_PATH","paths":["/home/oleg/src/guix-wigust","/tmp/n= oexist","/tmp/noexist","/tmp/noexist","/tmp/noexist","/tmp/noexist","/tmp/n= oexist","/tmp/noexist","/tmp/noexist","/tmp/noexist","/tmp/noexist","/tmp/n= oexist","/tmp/noexist","/tmp/noexist","/tmp/noexist","/tmp/noexist","/tmp/n= oexist","/tmp/noexist","/tmp/noexist"]} Initially the intent was to warn users that =E2=80=98GUIX_PACKAGE_PATH=E2= =80=99 is set and not captured in the output of =E2=80=98guix describe=E2=80=99, because = fundamentally it cannot be captured reliably. Thus, what about something as attached instead? Thanks, Ludo=E2=80=99. --=-=-= Content-Type: text/x-patch; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable diff --git a/guix/scripts/describe.scm b/guix/scripts/describe.scm index 7d0ecb0a4d..b6287d3a4c 100644 --- a/guix/scripts/describe.scm +++ b/guix/scripts/describe.scm @@ -1,5 +1,5 @@ ;;; GNU Guix --- Functional package management for GNU -;;; Copyright =C2=A9 2018 Ludovic Court=C3=A8s +;;; Copyright =C2=A9 2018, 2019 Ludovic Court=C3=A8s ;;; Copyright =C2=A9 2018 Oleg Pykhalov ;;; ;;; This file is part of GNU Guix. @@ -85,7 +85,9 @@ Display information about the channels currently in use.\= n")) (format #t "~%GUIX_PACKAGE_PATH=3D\"~a\"~%" string)) ('channels (format #t (G_ "~%;; warning: GUIX_PACKAGE_PATH=3D\"~a\"~%") - string)))))) + string)) + (_ + (warning (G_ "'GUIX_PACKAGE_PATH' is set but it is not captured~%"= ))))))) =20 (define (channel->sexp channel) `(channel --=-=-=--