[Top][All Lists]

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

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

From: swedebugia
Subject: [bug#34548] [PATCH 06/24] gnu: Add r-leaflet.
Date: Thu, 21 Feb 2019 21:03:08 +0100

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

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.
Sent from my k-9 mail for Android.

reply via email to

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