guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] Add ggplot2.


From: Ludovic Courtès
Subject: Re: [PATCH] Add ggplot2.
Date: Tue, 01 Sep 2015 23:26:02 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Howdy Ricardo!

Ricardo Wurmus <address@hidden> 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’.



reply via email to

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