guix-patches
[Top][All Lists]
Advanced

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

[bug#34548] [PATCH 06/24] gnu: Add r-leaflet.


From: Brett Gilio
Subject: [bug#34548] [PATCH 06/24] gnu: Add r-leaflet.
Date: Thu, 21 Feb 2019 14:14:18 -0600
User-agent: mu4e 1.0; emacs 26.1

swedebugia writes:

> Brett Gilio <address@hidden> skrev: (19 februari 2019 19:32:12 CET)
>>
>>Ricardo Wurmus writes:
>>
>>> Hi Brett,
>>>
>>>> From 26fbdbeee9ec293511648dc321abb09b6989fadc Mon Sep 17 00:00:00
>>2001
>>>> From: Brett Gilio <address@hidden>
>>>> Date: Mon, 18 Feb 2019 19:37:02 -0600
>>>> Subject: [PATCH 06/24] gnu: Add r-leaflet.
>>>>
>>>> * gnu/packages/cran.scm (r-leaflet): New variable.
>>>
>>> Unfortunately, this one is not okay.  It’s been on my list to add
>>> r-leaflet, but the package bundles minified JavaScript libraries.
>>>
>>> These sources of these libraries will need to be added as native
>>inputs
>>> to replace the bundled sources.  We can minify them with the
>>uglify-js
>>> package.
>>
>>Understood. I will make a note to look that and see if I can revise the
>>patch if that option seems formidable. However, on that note, do we
>>have
>>any progress on an npm importer? I know that came up awhile back.
>
> Julien is sitting on the patchset right now since a few weeks. 
>
> It still has some defects still but is mostly working.
>
> The problem of circular dependencies is still unsolved. I suggest we add to 
> our importer the ability to parse and select versions recursively. I tried 
> but failed to get it to work before I gave up.

Hey, where is the patchset located? I wouldn't mind taking a look.





reply via email to

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