[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#38576] [PATCH] gnu: r-irkernel: Fix R kernel loading
From: |
Roel Janssen |
Subject: |
[bug#38576] [PATCH] gnu: r-irkernel: Fix R kernel loading |
Date: |
Tue, 07 Jan 2020 22:02:13 +0100 |
User-agent: |
Evolution 3.34.2 (3.34.2-1.fc31) |
On Thu, 2020-01-02 at 12:43 +0100, Ludovic Courtès wrote:
> Hi,
>
> Lars-Dominik Braun <address@hidden> skribis:
>
> > > An argument in favor of the status quo would be that it allows users to
> > > choose between ‘r’ and ‘r-minimal’. Is that a compelling argument?
> > reading the documentation I thought this was possible using
> > --with-input=r-minimal=r ?
>
> Yes, good point.
>
> > > However, if we go that route, we should arrange to not propagate
> > > ‘r-minimal’ (it’s intrusive) and instead have ‘kernel.json’ do the right
> > > thing.
> > I’m not following, sorry. What do you suggest kernel.json should do?
>
> I was suggesting hard-coding the file name of the ‘R’ executable in
> ‘kernel.json’, but I see you already did that in your initial patch.
> Sorry for the confusion.
>
> On second thought, I think propagating R is acceptable in this case
> because a Jupyter kernel is a thin wrapper around a programming language
> implementation.
>
> Unless there are objections, I’ll apply your initial patch.
>
I'm too late, but doesn't this break the kernel for people who have the "full" R
in their profile, and therefore expect the "full" R to be available in a Jupyter
notebook?
Kind regards,
Roel Janssen