[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.