From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:38539) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jHbPv-0005T8-Vi for guix-patches@gnu.org; Thu, 26 Mar 2020 18:56:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1jHbPu-0005dO-HB for guix-patches@gnu.org; Thu, 26 Mar 2020 18:56:03 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:54372) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1jHbPu-0005dI-Dd for guix-patches@gnu.org; Thu, 26 Mar 2020 18:56:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jHbPu-0004JX-Cf for guix-patches@gnu.org; Thu, 26 Mar 2020 18:56:02 -0400 Subject: [bug#39765] Add package JupyterLab Resent-Message-ID: From: Ludovic =?UTF-8?Q?Court=C3=A8s?= References: <20200224101810.GA9010@zpidnp36> Date: Thu, 26 Mar 2020 23:55:26 +0100 In-Reply-To: <20200224101810.GA9010@zpidnp36> (Lars-Dominik Braun's message of "Mon, 24 Feb 2020 11:18:10 +0100") Message-ID: <87d08y915t.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable 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: Lars-Dominik Braun Cc: 39765@debbugs.gnu.org Hi Lars, Sorry for the late reply. Lars-Dominik Braun skribis: > this patch series adds Jupyter=E2=80=99s JupyterLab, which is the new fro= ntend for > Jupyter Notebooks. The software works fine, but there are a few caveats > > 1) it comes with bundled pre-compiled JavaScript, which cannot be removed= until > we have proper support for importing from NPM > 2) it contains an extension manager, that downloads arbitrary packages fr= om NPM > (`jupyter lab build`). This works, but is less than optimal imo. We sh= ould > figure out how to package extensions in guix. > 3) also it is required to install the package `jupyter`, otherwise instal= led > kernels cannot be found and the `jupyter` command does not work. #2 should be quite easy to address: we could arrange to have that feature disabled by default, so that users don=E2=80=99t find themselves unknowingly downloading arbitrary code from npm. #3 is OK. #1 is a showstopper. :-/ I suppose that=E2=80=99s a lot of code that would need to be imported from npm, right? It=E2=80=99s sad because all this is free software, but we practically can= =E2=80=99t get the corresponding source. I=E2=80=99ve pushed the first two patches of the series (python-json5 and python-pytest-check-links). Comments on the other bits that are readily applicable: >>>From a47fd94aa6f3e62b77f3b7208c4e6757e3a9ee08 Mon Sep 17 00:00:00 2001 > From: Lars-Dominik Braun > Date: Thu, 12 Dec 2019 08:53:39 +0100 > Subject: [PATCH 5/5] gnu: python-notebook: Support UNIX domain sockets > > * gnu/packages/python-xyz.scm (python-notebook): Add patch from upstream > https://github.com/jupyter/notebook/pull/4835 > (python-requests-unixsocket) New variable > --- > ...pyter-unix-domain-sockets-4835-5.7.4.patch | 591 ++++++++++++++++++ > gnu/packages/python-xyz.scm | 35 +- > 2 files changed, 624 insertions(+), 2 deletions(-) > create mode 100644 gnu/packages/patches/jupyter-unix-domain-sockets-4835= -5.7.4.patch > > diff --git a/gnu/packages/patches/jupyter-unix-domain-sockets-4835-5.7.4.= patch b/gnu/packages/patches/jupyter-unix-domain-sockets-4835-5.7.4.patch > new file mode 100644 > index 0000000000..134d3ad2b8 > --- /dev/null > +++ b/gnu/packages/patches/jupyter-unix-domain-sockets-4835-5.7.4.patch > @@ -0,0 +1,591 @@ Please add provenance info at the top of the patch (such as the URL of the upstream commit), as well as a line or two explaining what it does. You can omit =E2=80=9C-4835-5.7.4=E2=80=9D from the file name. Make sure to add the file to =E2=80=98gnu/local.mk=E2=80=99. That said, it=E2=80=99s a big patch, so it would be even better if we didn= =E2=80=99t have to carry it. Will the next version of =E2=80=98notebook=E2=80=99 incl= ude it? Last, =E2=80=98python-requests-unixsocket=E2=80=99 should be added in a sep= arate patch. [...] > + (arguments > + ;; tests depend on very specific package version, which are not ava= ilable in guix > + '(#:tests? #f)) Perhaps add a =E2=80=9CFIXME=E2=80=9D and clarify which packages we=E2=80= =99re talking about (the =E2=80=9Cnot available=E2=80=9D bit is bound to become outdated :-)). > + (home-page > + "https://github.com/msabramo/requests-unixsocket") > + (synopsis > + "Use requests to talk HTTP via a UNIX domain socket") > + (description > + "Use requests to talk HTTP via a UNIX domain socket") Please follow the synopsis/description guidelines (info "(guix) Synopses and Descriptions"). Thank you for this endeavor! Ludo=E2=80=99.