[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#38576] [PATCH] gnu: r-irkernel: Fix R kernel loading

From: Ludovic Courtès
Subject: [bug#38576] [PATCH] gnu: r-irkernel: Fix R kernel loading
Date: Wed, 08 Jan 2020 00:01:26 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux)

Hi Roel,

Roel Janssen <address@hidden> skribis:

> 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?

Indeed, it forces ‘r-minimal’ for use in the kernel.  But there are
workarounds, as Lars-Dominik suggests above.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]