Mailing-List: contact dev-help@mxnet.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@mxnet.incubator.apache.org
MIME-Version: 1.0
In-Reply-To: <CAOKkJGq_zC54_yLdi2AfQnh0ZB2zjh+X9ATGVSSFQPnNwLP5Mw@mail.gmail.com>
References: <CALGG8z1NsbrGEPU2Wq+Obi0f4fG55UZcFQDFMKjebysO15sZMA@mail.gmail.com>
 <DM5PR04MB0747D12D190FA98DA00B1C9491D50@DM5PR04MB0747.namprd04.prod.outlook.com>
 <CAJ_LAMCJ9+oG0z7BFR7535h3hw-Wk165BHmkA3R4pW0fXyxZuw@mail.gmail.com>
 <DM5PR04MB07477F013FD68A922CD0EBC091AA0@DM5PR04MB0747.namprd04.prod.outlook.com>
 <CAJ_LAMBvt9q3xegR0YUzb8HFvgGstcXNRWzO3Y0=O7SwNFgYfg@mail.gmail.com>
 <CALGG8z09RAsatV9NkQLMjf+imXJmjqtVm9zQj6seafTnXmaaaQ@mail.gmail.com>
 <DM5PR04MB07472C4BE8D6BF4B03FA56EA91AA0@DM5PR04MB0747.namprd04.prod.outlook.com>
 <CAJ_LAMCGPoY0z4Rdm36KSEwumy1PtnubY8s5LEZ6TF5F249yYA@mail.gmail.com>
 <DM5PR04MB0747DC6A720E42C5FF9D3A0091AA0@DM5PR04MB0747.namprd04.prod.outlook.com>
 <CAJ_LAMD0hfAyP5QE8eH2vL85azwCFYQ0FGRxJHApdAJpTEb_Pw@mail.gmail.com>
 <CAOKkJGq9BOG24JDcBcu7Tm1kJZZH5qzNY_XcLLn6n8wXkpLSzg@mail.gmail.com>
 <CAJ_LAMBnPdtt5nM8c8=0o2rRSeR9ria89kMBQ51cBXid9eZN=A@mail.gmail.com> <CAOKkJGq_zC54_yLdi2AfQnh0ZB2zjh+X9ATGVSSFQPnNwLP5Mw@mail.gmail.com>
From: Naveen Swamy <mnnaveen@gmail.com>
Date: Tue, 11 Jul 2017 20:21:12 -0700
Message-ID: <CAAy-JxDRgoThwL_tPfNreiY2q6B-Q8dBrf-H9tS5DaEzGAnqMQ@mail.gmail.com>
Subject: Re: Rcpp licensing in Apache MXNet
To: dev@mxnet.incubator.apache.org
Content-Type: multipart/alternative; boundary="001a113a7af041f5ea05541651c5"
archived-at: Wed, 12 Jul 2017 03:58:59 -0000

--001a113a7af041f5ea05541651c5
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Ly,
     Can we skip R pkg for the proposed release as KK mentioned and add
it/alter based on the advice we get from ASF legal?

---KK Says---
As I understand, if we skip the R pkg when releasing the a new version of
MXNet, everything is OK. This is can be done by adding a .gitattribute.
---

others,
     thoughts/concerns?

Thanks, Naveen


On Tue, Jul 11, 2017 at 3:56 PM, Ly Nguyen <nguyenlyx@gmail.com> wrote:

> Hey KK,
>
> I know we're planning a release end of this week/beginning of next week. =
It
> may be critical to get this cleared if it is an issue. Eager to hear back=
.
> :)
>
> On Tue, Jul 11, 2017 at 3:35 PM, Qiang Kou <qkou@umail.iu.edu> wrote:
>
> > Hi, Ly,
> >
> > I will let you know when I have the answer.
> >
> > Best,
> >
> > KK
> >
> > On Tue, Jul 11, 2017 at 10:50 AM, Ly Nguyen <nguyenlyx@gmail.com> wrote=
:
> >
> > > Hi @KK, any updates from legal on whether excluding the R pkg is a
> > solution
> > > for our next release?
> > >
> > > On Mon, Jul 10, 2017 at 10:49 AM, Qiang Kou <qkou@umail.iu.edu> wrote=
:
> > >
> > > > Thank you for the info.
> > > >
> > > > As I understand, if we skip the R pkg when releasing the a new
> version
> > of
> > > > MXNet, everything is OK. This is can be done by adding a
> .gitattribute.
> > > >
> > > > I will ask on legal-discuss@ for more info and confirmation.
> > > >
> > > > Really thank you for all the info! It is super helpful.
> > > >
> > > > Best,
> > > >
> > > > KK
> > > >
> > > >
> > > > On Fri, Jul 7, 2017 at 8:47 AM, Felix Cheung <
> > felixcheung_m@hotmail.com>
> > > > wrote:
> > > >
> > > > > I was only referring to string_hash_code.c - it's not being built
> and
> > > > it's
> > > > > not part of the binaries release.
> > > > >
> > > > > There are two parts to it.
> > > > >
> > > > > For Spark binaries release, R package is built and the output is
> > > packaged
> > > > > along with the rest of all jars  and python stuff.
> > > > >
> > > > > There is also a source-only R package that we want to publish to
> > CRAN.
> > > > > This contains only R source (no java stuff). CRAN will then build
> > cross
> > > > > platform from that source - but again the part with
> > string_hash_code.c
> > > is
> > > > > disabled.
> > > > >
> > > > > I guess we should have removed string_hash_code.c from source but
> we
> > > are
> > > > > secretly hoping we could sort that out at some point in the
> future..
> > > (ie.
> > > > > building cross platform)
> > > > >
> > > > >
> > > > > _____________________________
> > > > > From: Qiang Kou <qkou@umail.iu.edu<mailto:qkou@umail.iu.edu>>
> > > > > Sent: Friday, July 7, 2017 8:34 AM
> > > > > Subject: Re: Rcpp licensing in Apache MXNet
> > > > > To: <dev@mxnet.incubator.apache.org<mailto:dev@mxnet.
> > > > incubator.apache.org
> > > > > >>
> > > > >
> > > > >
> > > > > Really thank you for the info.
> > > > >
> > > > > Can you tell us a little more on how Spark handles the R package?
> > > > >
> > > > > The building of R package is skipped when releasing, right?
> > > > >
> > > > > Best wishes,
> > > > >
> > > > > KK
> > > > >
> > > > > On Fri, Jul 7, 2017 at 7:41 AM, Felix Cheung <
> > > felixcheung_m@hotmail.com<
> > > > > mailto:felixcheung_m@hotmail.com>>
> > > > > wrote:
> > > > >
> > > > > > Re this
> > > > > >
> > > > > > The Spark R package includes "R.h" and "Rinternals.h", which ar=
e
> > both
> > > > > LGPL
> > > > > > (
> > > > > > https://github.com/apache/spark/blob/master/R/pkg/src-
> > > > > > native/string_hash_code.c#L24-L25
> > > > > > ).
> > > > > >
> > > > > > This is actually disabled - it is not getting built and it is n=
ot
> > in
> > > > the
> > > > > > release - because the way SparkR is packaged in Spark release w=
e
> > > > haven't
> > > > > > figured out how to release cross multiple platforms.
> > > > > >
> > > > > > (I'm one of the main contributor to SparkR)
> > > > > >
> > > > > > ________________________________
> > > > > > From: Henri Yandell <bayard@apache.org<mailto:bayard@apache.org
> >>
> > > > > > Sent: Friday, July 7, 2017 12:02:16 AM
> > > > > > To: dev@mxnet.incubator.apache.org<mailto:dev@mxnet.incubator.
> > > > apache.org
> > > > > >
> > > > > > Subject: Re: Rcpp licensing in Apache MXNet
> > > > > >
> > > > > > We should bring this up on general@incubator or legal-discuss@.
> > > > > >
> > > > > > On Thu, Jul 6, 2017 at 10:59 PM, Qiang Kou <qkou@umail.iu.edu
> > > <mailto:
> > > > > qkou@umail.iu.edu>> wrote:
> > > > > >
> > > > > > > Really thank you for the clearance.
> > > > > > >
> > > > > > > From the link you provided,
> > > > > > >
> > > > > > > "However, if the component is only needed for optional
> features,
> > a
> > > > > > project
> > > > > > > can provide the user with instructions on how to obtain and
> > install
> > > > the
> > > > > > > non-included work."
> > > > > > >
> > > > > > > I think this is exactly our case. We provide instructions on
> how
> > to
> > > > > > install
> > > > > > > GNU R and the Rcpp package.
> > > > > > >
> > > > > > > Besides, our case is very similar with Spark.
> > > > > > >
> > > > > > > The Spark R package includes "R.h" and "Rinternals.h", which
> are
> > > both
> > > > > > LGPL
> > > > > > > (
> > > > > > > https://github.com/apache/spark/blob/master/R/pkg/src-
> > > > > > > native/string_hash_code.c#L24-L25
> > > > > > > ).
> > > > > > >
> > > > > > > Best wishes,
> > > > > > >
> > > > > > > Qiang Kou
> > > > > > >
> > > > > > > On Thu, Jul 6, 2017 at 6:09 PM, Felix Cheung <
> > > > > felixcheung_m@hotmail.com<mailto:felixcheung_m@hotmail.com>>
> > > > > > > wrote:
> > > > > > >
> > > > > > > > I'm not the authority on this but a release from an ASF
> project
> > > is
> > > > > > > > different from releasing with the Apache 2.0 license.
> > > > > > > >
> > > > > > > > You can find more out this here
> > > > > > > > https://www.apache.org/legal/resolved.html#optional
> > > > > > > >
> > > > > > > > Specifically, quote:
> > > > > > > > "Optional means that the component is not required for
> standard
> > > use
> > > > > of
> > > > > > > the
> > > > > > > > product or for the product to achieve a desirable level of
> > > > quality."
> > > > > > > >
> > > > > > > > Basically my interpretation is that if an ASF project canno=
t
> > > > function
> > > > > > > > without a component, then it is required and such required
> > > > component
> > > > > > > cannot
> > > > > > > > be one of the prohibited licenses.
> > > > > > > >
> > > > > > > > https://www.apache.org/legal/resolved.html#category-x
> > > > > > > >
> > > > > > > > Probably a good idea to start a thread with legal@ if there
> > are
> > > > more
> > > > > > > > questions.
> > > > > > > >
> > > > > > > > _____________________________
> > > > > > > > From: Qiang Kou <qkou@umail.iu.edu<mailto:qkou@umail.iu.edu
> > > > ><mailto:
> > > > > qkou@umail.iu.edu>>
> > > > > > > > Sent: Thursday, July 6, 2017 3:41 PM
> > > > > > > > Subject: Re: Rcpp licensing in Apache MXNet
> > > > > > > > To: <dev@mxnet.incubator.apache.org<mailto:dev@mxnet.
> > > > > incubator.apache.org><mailto:dev@mxnet.
> > > > > > > incubator.apache.org<http://incubator.apache.org>
> > > > > > > > >>
> > > > > > > >
> > > > > > > >
> > > > > > > > Hi, Felix,
> > > > > > > >
> > > > > > > > I just want to make sure there is no misunderstanding betwe=
en
> > us.
> > > > > > > >
> > > > > > > > (1) The MXNet R package heavily used the Rcpp package, but =
we
> > > only
> > > > > > > include
> > > > > > > > the
> > > > > > > > header file, Rcpp.h, like https://github.com/dmlc/mxnet/
> > > > > > > > blob/master/R-package/src/base.h#L11
> > > > > > > >
> > > > > > > > The MXNet R package doesn't contain any source code from Rc=
pp
> > and
> > > > > there
> > > > > > > is
> > > > > > > > no linking against Rcpp.
> > > > > > > >
> > > > > > > > (2) There is "LinkingTo: Rcpp" in the R package description=
 (
> > > > > > > > https://github.com/dmlc/mxnet/blob/master/R-package/
> > > > DESCRIPTION#L32
> > > > > ).
> > > > > > > > However, this only means we want to use the header files fr=
om
> > > Rcpp.
> > > > > > > >
> > > > > > > > "A package that wishes to make use of header files in other
> > > > > > > > packages needs to declare them as a
> > > > > > > > comma-separated list in the field =E2=80=98LinkingTo=E2=80=
=99 in the
> > DESCRIPTION
> > > > > file.
> > > > > > "
> > > > > > > >
> > > > > > > > More details can be found from R's official manual (
> > > > > > > > https://cran.r-project.org/doc/manuals/r-release/R-exts.
> > > > > > > > html#Package-Dependencies).
> > > > > > > >
> > > > > > > > (3) Many R packages who use Rcpp to communicate with C++ ha=
ve
> > > been
> > > > > > > released
> > > > > > > > under Apache License, like feather (https://github.com/wesm=
/
> > > > > > > > feather/blob/master/R/DESCRIPTION).
> > > > > > > >
> > > > > > > > Best,
> > > > > > > >
> > > > > > > > Qiang Kou
> > > > > > > >
> > > > > > > > On Thu, Jul 6, 2017 at 1:20 AM, Felix Cheung <
> > > > > > felixcheung_m@hotmail.com<mailto:felixcheung_m@hotmail.com><
> > > > > > > > mailto:felixcheung_m@hotmail.com>>
> > > > > > > > wrote:
> > > > > > > >
> > > > > > > > > On a related note, Rcpp, used extensively in the R packag=
e,
> > is
> > > > > > > > GPLv2/GPLv3
> > > > > > > > > licensed.
> > > > > > > > >
> > > > > > > > > I'm not aware of any other R package available that
> supports
> > > > > R<->C++.
> > > > > > > > >
> > > > > > > > > https://www.apache.org/legal/resolved.html#category-x
> > > > > > > > > https://cran.r-project.org/web/packages/Rcpp/index.html
> > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Qiang Kou
> > > > > > > qkou@umail.iu.edu<mailto:qkou@umail.iu.edu>
> > > > > > > School of Informatics and Computing, Indiana University
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Qiang Kou
> > > > > qkou@umail.iu.edu<mailto:qkou@umail.iu.edu>
> > > > > School of Informatics and Computing, Indiana University
> > > > >
> > > > >
> > > > >
> > > >
> > > >
> > > > --
> > > > Qiang Kou
> > > > qkou@umail.iu.edu
> > > > School of Informatics and Computing, Indiana University
> > > >
> > >
> >
> >
> >
> > --
> > Qiang Kou
> > qkou@umail.iu.edu
> > School of Informatics and Computing, Indiana University
> >
>

--001a113a7af041f5ea05541651c5--