[Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Christian Authmann
Hello, I've developed a solution to integrate R into an application in a safer way than the direct integration provided by RInside. It is conceptually similar to RServe: the application does not link to R, but communicates with a different process where R is running. Securing the separate pro

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Dirk Eddelbuettel
On 20 November 2014 at 14:58, Christian Authmann wrote: | Hello, | | I've developed a solution to integrate R into an application in a safer | way than the direct integration provided by RInside. It is conceptually | similar to RServe: the application does not link to R, but communicates | wit

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Smith, Dale (Norcross)
I've always had licensing concerns with RInside. I preferred Rserve and it worked well for our purposes years ago. This new code sounds like it is more flexible than Rserve and deserves to be a separate package, not in RInside. But that's a personal preference to avoid the *appearance* of licen

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Dirk Eddelbuettel
On 20 November 2014 at 14:28, Smith, Dale (Norcross) wrote: | I've always had licensing concerns with RInside. I preferred Rserve and it worked well for our purposes years ago. | | This new code sounds like it is more flexible than Rserve and deserves to be a separate package, not in RInside. B

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Smith, Dale (Norcross)
I never distributed my work and don't now. Dale Smith, Ph.D. Senior Financial Quantitative Analyst Financial & Risk Management Solutions Fiserv Office: 678-375-5315 www.fiserv.com FORTUNE® Magazine's 2014 World's Most Admired Companies Facebook: Like Fiserv  ·  Twitter: Follow @Fiserv  ·  LinkedI

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Dirk Eddelbuettel
On 20 November 2014 at 15:25, Smith, Dale (Norcross) wrote: | I never distributed my work and don't now. Same with what I do at work (as opposed to what I do as a private person in the Open Source commons). And I don't talk about what I do at work either. So your entire point of what _Christi

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Smith, Dale (Norcross)
I agree the issue is moot and resulted from my own confusion. I just wanted to make things perfectly clear - I have never, nor do I intend to distribute. My work with R, Rcpp, etc, is internal use only. I think we can drop this now. Dale Smith, Ph.D. Senior Financial Quantitative Analyst Financ

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Christian Authmann
Hello, I don't want to get into lengthy legal discussions (I'm a programmer, not a lawyer), but let me just state my intent. From the way I read the GPL-FAQ > http://www.gnu.org/licenses/gpl-faq.html the GPL2 only applies to derivative work and linking. Which means that the client, which is w

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Dirk Eddelbuettel
Hi Christian, On 20 November 2014 at 17:10, Christian Authmann wrote: | Hello, | | I don't want to get into lengthy legal discussions (I'm a programmer, | not a lawyer), but let me just state my intent. | | From the way I read the GPL-FAQ | > http://www.gnu.org/licenses/gpl-faq.html | the GP

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Christian Authmann
On 11/20/2014 05:41 PM, Dirk Eddelbuettel wrote: > Does the client code use R code to (de-)serialize R objects? It's right in the code :-) TL;DR: It doesn't. The code only serializes generic types (int, std::string, std::vector, ..) and user-defined classes with their own serialization methods,

Re: [Rcpp-devel] Review request: Sandboxed R integration via RInside

2014-11-20 Thread Dirk Eddelbuettel
On 20 November 2014 at 19:04, Christian Authmann wrote: | > I would really much prefer if you could alter the pull request and change the | > licensing. | | Can do for the final pull request after review. I think I would like to change that in the pull request. Code review is light. It is all