Mailing-List: contact general-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: general@incubator.apache.org
MIME-Version: 1.0
In-Reply-To: <1D2BB137-2711-4E2F-BB97-6F670D73073A@classsoftware.com>
References: <CAMBJEmWiaOpprh7X=ntfrzKcJtLRHXPGwbtiU03+KkP5JxDKxA@mail.gmail.com>
 <CAMBJEmUZCKYCZCWHXLZTSaGRfV+LNfUtdiOtk2sGv4Hgcy6qsQ@mail.gmail.com> <1D2BB137-2711-4E2F-BB97-6F670D73073A@classsoftware.com>
From: Stian Soiland-Reyes <stain@apache.org>
Date: Mon, 20 Jun 2016 16:06:51 +0100
Message-ID: <CAMBJEmUPi2JwykzXJ59RfAb-R90MEfdyh4AFPDhtHVHyGbeEuA@mail.gmail.com>
Subject: Re: [VOTE] Release Apache Taverna Command-line Tool 3.1.0-incubating RC3
To: general@incubator.apache.org
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
archived-at: Mon, 20 Jun 2016 15:07:15 -0000

On 18 June 2016 at 01:50, Justin Mclean <justin@classsoftware.com> wrote:
> Hi,
>
> +1 (binding). Can you fix the LICENSE and NOTICE issues in the next relea=
se please.

Thank you - already fixed in master. Getting smarter every day..


> I can help by reviewing the release but I=E2=80=99m at my limit mentor wi=
se sorry.

Always appreciated - thanks!


> - able to compile all from source but it seems the order matters (i.e. yo=
u need to compile the engine first) you may want to mention that in README =
if it=E2=80=99s not already

Of course once these are in Maven Central it won't be a big blocker in the =
end.

However I agree the build order should be more explicit, so I have
added for the future:

https://github.com/apache/incubator-taverna-engine#prerequisites
https://github.com/apache/incubator-taverna-common-activities#prerequisites
https://github.com/apache/incubator-taverna-commandline#prerequisites

(Note that the linked download pages for engine/common-activities are
only on http:// taverna.staging.apache.org/ for now)


> In apache-taverna-commandline-3.1.0-incubating
> - wording in NOTICE is is a little odd but probably fine.

I guess you mean this section?

> Portions of this software were originally based on the following:
> - Copyright 2010-2014 University of Manchester, UK
> These have been licensed to the Apache Software Foundation under a softwa=
re grant.

This is how it was suggested to us by our mentors; but we're open for
improvements - this text is the same across our NOTICEs (however the
start-year might vary depending on the code base).

Semantically I guess the software is not based on a copyright.. :)

How about:

> Portions of this software is:
> - Copyright 2010-2014 University of Manchester, UK
> These have been licensed to the Apache Software Foundation under a softwa=
re grant.

?


> - year in NOTICE needs updating

Bah - thanks. Updated. :)


> In apache-taverna-common-activities-2.1.0-incubating
> - W3C license info should be moved to LICENSE or preferably to it own fil=
e and a pointer to that file added [1]

Done - https://issues.apache.org/jira/browse/TAVERNA-986


> - LICENSE is missing license for this file [3]

Oops! Well spotted. It is Public Domain, so there is obviously no
requirement to propagate any license,

But is "Public Domain" valid outside US?  Should we append ASF headers
on it? (That should be allowed if it's PD.. at least if that is done
by an USAnian)


Anyway, for information purposes I've added to top-level LICENSE
---------------------------------------------------------
./taverna-interaction-activity/src/main/resources/json2.js
is Public Domain, see https://github.com/douglascrockford/JSON-js/
and http://www.json.org/js.html
-------------------------------------------------------


> In apache-taverna-engine-3.1.0-incubating
> - The licence of this file [2] needs to be mentioned in LICENSE not NOTIC=
E [1]. A pointer to the license file is preferred.

Done. https://issues.apache.org/jira/browse/TAVERNA-985

(Also as you suggested I put it in META-INF/LICENSE rather than META-INF/NO=
TICE)


--=20
Stian Soiland-Reyes
Apache Taverna (incubating), Apache Commons
http://orcid.org/0000-0001-9842-9718

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org