Return-Path: 
 <cassandra-dev-return-885-apmail-incubator-cassandra-dev-archive=incubator.apache.org@incubator.apache.org>
Delivered-To: apmail-incubator-cassandra-dev-archive@minotaur.apache.org
Received: (qmail 95906 invoked from network); 9 Sep 2009 16:24:20 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3)
  by minotaur.apache.org with SMTP; 9 Sep 2009 16:24:20 -0000
Received: (qmail 93520 invoked by uid 500); 9 Sep 2009 16:24:20 -0000
Delivered-To: apmail-incubator-cassandra-dev-archive@incubator.apache.org
Received: (qmail 93339 invoked by uid 500); 9 Sep 2009 16:24:19 -0000
Mailing-List: contact cassandra-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:cassandra-dev-help@incubator.apache.org>
List-Unsubscribe: <mailto:cassandra-dev-unsubscribe@incubator.apache.org>
List-Post: <mailto:cassandra-dev@incubator.apache.org>
List-Id: <cassandra-dev.incubator.apache.org>
Reply-To: cassandra-dev@incubator.apache.org
Delivered-To: mailing list cassandra-dev@incubator.apache.org
Received: (qmail 93175 invoked by uid 99); 9 Sep 2009 16:24:19 -0000
Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136)
    by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 09 Sep 2009 16:24:19 +0000
X-ASF-Spam-Status: No, hits=-0.0 required=10.0
	tests=SPF_PASS
X-Spam-Check-By: apache.org
Received-SPF: pass (athena.apache.org: domain of sebbaz@gmail.com designates
 209.85.219.210 as permitted sender)
Received: from [209.85.219.210] (HELO mail-ew0-f210.google.com)
 (209.85.219.210)
    by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 09 Sep 2009 16:24:11 +0000
Received: by ewy6 with SMTP id 6so119739ewy.27
        for <multiple recipients>; Wed, 09 Sep 2009 09:23:49 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=gamma;
        h=domainkey-signature:mime-version:received:in-reply-to:references
         :date:message-id:subject:from:to:cc:content-type;
        bh=+SE6jPJthDLnLs8K8FmyiT4SNYYUwrxlvONNdIw3yFQ=;
        b=qe2YYL4/D1bECJ4kx8V9uXiaHPBbOlr7fZ4C4kzK2Yrp3D+CbfiTHrDXzuR1FCPkb5
         rjFmpn1jC+gOo/RMbEhvV6/XGUAuO+K8FoANYOxsX0jhDvq8XYXc+3O1Dr97kBkyAmFT
         W1d8SIUSbpYdmNODlFnTQe+2/NysaOYnIKaXA=
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :cc:content-type;
        b=Hf7CZd7FsNzIjThIrpHJMY27Fkuse/4hRwVZVY/FjJ9xH60C0xBY5EVDOmQCDJS3sf
         rS1OVRJdY4pE1rkOSKHZV3T5PAeRn6tgXVszamlkDCXTkOLtz+NPGtJhK06dd4bo6hRu
         3kLisvkIP+4+TnklU/OkLlADPmuYc0CWkltYg=
MIME-Version: 1.0
Received: by 10.216.86.85 with SMTP id v63mr105541wee.32.1252513429653; Wed,
	09 Sep 2009 09:23:49 -0700 (PDT)
In-Reply-To: <1252509435.6875.190.camel@achilles>
References: <1252440588.6875.157.camel@achilles>
	 <25aac9fc0909081428g1c34b4d8ifad53f6e09b7c740@mail.gmail.com>
	 <1252509435.6875.190.camel@achilles>
Date: Wed, 9 Sep 2009 17:23:49 +0100
Message-ID: <25aac9fc0909090923ub1e5bd8j5065b9d062afd3cf@mail.gmail.com>
Subject: Re: Review of cassandra NOTICE and LICENSE requested.
From: sebb <sebbaz@gmail.com>
To: general@incubator.apache.org
Cc: "cassandra-dev@incubator.apache.org" <cassandra-dev@incubator.apache.org>
Content-Type: text/plain; charset=ISO-8859-1
X-Virus-Checked: Checked by ClamAV on apache.org

On 09/09/2009, Eric Evans <eevans@rackspace.com> wrote:
> On Tue, 2009-09-08 at 22:28 +0100, sebb wrote:
>  > Does the release actually *include* JUnit, or is that only used for
>  > testing? If so, then it should not be the in the NOTICE file, nor in
>  > the binary release.
>
>
> We have a source and binary distribution with the same NOTICE and
>  LICENSE texts in both artifacts. While we don't want/need to ship the
>  junit jar in the binary distribution (see CASSANDRA-417), shipping it in
>  the source artifact is intentional.
>

That's not normal; the source artifact should only contain files which
are in SVN and the JUnit jar should not be in SVN. This is
particularly true since you are using Maven which will fetch any
required dependencies when doing a build.

>  > Is ANTLR actually needed at run-time, or is it only needed for
>  > building the code?
>
>
> It's only used to build.
>
>
>  > I'm not sure whether it needs to be mentioned in NOTICE if it is only
>  > used at build-time.
>
>
> I had thought it was needed when redistributing code with the terms that
>  require it (and as far as I know, that's only Apache License 2.0). I was
>  told on this list (repeatedly), that I was wrong. Now I'm just confused.
>  Again.
>
>
>  > It should not be in the binary release if it's not needed at run-time.
>
>
> Agreed (CASSANDRA-417), but again, shipping a copy in the source
>  artifact is intentional, (hence the entries in NOTICE and LICENSE).

Same as for JUnit.

>
>  > The NOTICE file mentions SLF4J, which AIUI is a logging abstraction -
>  > does the binary release include a logging implementation? If so, it
>  > will need to be documented in the LICENSE and NOTICE.
>
>
> We use log4j for logging, but we require/ship thrift which in turn
>  requires slf4j.
>
>
>  > >  It would be great if we could get these reviewed ahead of time so
>  > that
>  > >  any feedback could be incorporated before the next release vote.
>  >
>  > It's not possible to be definitive about the files without knowing
>  > what libraries are actually going to be included in the binary
>  > release.
>
>
> There haven't been any changes since the last review, (see
>  https://svn.apache.org/repos/asf/incubator/cassandra/trunk/lib).

I would not expect to see any jars in SVN for a Maven project.

>  Thanks for taking the time to look at this.
>
>
>  --
>
> Eric Evans
>  eevans@rackspace.com
>
>
>  ---------------------------------------------------------------------
>  To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>  For additional commands, e-mail: general-help@incubator.apache.org
>
>