Re: [PATCH] Add ggplot2.

[ludo@gnu.org (Ludovic Courtès)]

Howdy Ricardo!

Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> skribis:

> while writing the r-build-system and the CRAN importer I tested it by
> packaging ggplot2 with all dependencies.  16 patches are attached; all
> of them are very simple (this bodes well for the future of R on Guix).

Indeed, that’s a really good sign.

> To make sure ggplot2 actually works I did this:
>
>     guix package -p ~/r-tests -i r r-ggplot2
>     export R_LIBS_SITE="$HOME/r-tests/site-library/"
>     ~/r-tests/bin/R
>
>     > data(iris)
>     > library(ggplot2)
>     > ggplot(iris,aes(Sepal.Length,Sepal.Width)) + geom_point()
>
>     (produces a scatter plot.)

Neat.

I look at the patches briefly and don’t have anything to say.  Perhaps
the only thing that might be worth checking is whether license info is
usually accurate, but I suppose pretty much all R packages are GPLv3+ in
practice since that’s what R is under?

So maybe just double-check, change the licenses to GPLv3+, and push.

Thank you,
Ludo’.