> On May 27, 2019, at 00:11, Vinayakumar B <[email protected]> wrote:
> 
> 1. I can see 'incubating' in only archives' names.
> 
> Shouldn't 'incubating' also to be added to version of all artifacts?
> 
> I am not seeing specific mention of this in release policy, but other
> podlings followed this way.
> Since these artifacts individually can be downloaded via maven central,
> basically it should be part of artifacts version if I am not mistaken.

I checked other projects, especially Cassandra, and they did not have 
incubating in their version. I’d rather avoid that quandary.
> 
> 2. README refers to non existent version. Please correct.

Fixed in master. Will be fixed in next release.
> 
> 
> 3. Location of archives shared below.. should be inside
> 'dev/incubator/tuweni/' directory instead of 'dev/tuweni'. After release it
> will be moved to 'release' accordingly.
OK.
> 
> 
> -Vinay
>> On Sun, 26 May 2019, 8:34 am Antoine Toulme, <[email protected]> wrote:
>> 
>> Casting my +1 as well. Thanks Larry for looking into it.
>> 
>>> On May 25, 2019, at 7:54 PM, Antoine Toulme <[email protected]> wrote:
>>> 
>>> 
>>> 
>>>> On May 25, 2019, at 7:35 PM, larry mccay <[email protected]
>> <mailto:[email protected]>> wrote:
>>>> 
>>>> Built from source and ran tests.
>>>> I checked NOTICE and LICENSE files and have the following questions:
>>>> 
>>>> 1. Why does NOTICE include the following:
>>>> 
>>>> "In addition, this product includes software developed by
>>>> Copyright 2018-2019 ConsenSys, Inc.”
>>> I followed what I found in the NOTICE file of Apache Trafodion:
>>> https://github.com/apache/trafodion/blob/master/NOTICE <
>> https://github.com/apache/trafodion/blob/master/NOTICE>
>>> 
>>>> 
>>>> Is there code that is being bundled from ConsenSys or has it all been
>>>> contributed to ASF?
>>> All contributed. So maybe that’s not worth mentioning?
>>>> 
>>>> 2. Should we be referencing licenses from dependencies in the NOTICE
>> file?
>>> Yes, it’s mentioned here:
>>> 
>>> Please consult the list of dependencies under the dependency report
>> under the licenses folder.
>>> And the licenses folder contains the dependency report with the detail
>> of each dependency and their licenses.
>>> 
>>>> 
>>>> 3. Shouldn't the LICENSE file actually have the text for all the
>>>> dependencies licenses such as MIT?
>>> No idea.
>>>> 
>>>> 4. Should the separate release module for the Gossip app have a source
>>>> artifact? Apache releases are of source.
>>>> Any binary release is really a convenience.
>>>> I do see that the Gossip app is in the source of Tuweni itself - so that
>>>> may be enough.
>>> Yes, gossip sources are bundled with sources.
>>>> 
>>>> These are questions from comparing to other projects that I am involved
>>>> with and may not be strict rules.
>>>> I would refer you to [1] for details on NOTICE and LICENSE files.
>>> I’m not seeing the URL for [1]?
>>>> 
>>>> If there are actual issues with the above, I don't believe that they are
>>>> release blockers and that they can be fixed in a future release.
>>>> 
>>>> Here is my +1.
>>>> 
>>>> 
>>>> 
>>>> On Fri, May 24, 2019 at 9:55 AM Antoine Toulme <[email protected]
>> <mailto:[email protected]>> wrote:
>>>> 
>>>>> The tag is here:
>>>>> https://github.com/apache/incubator-tuweni/releases/tag/v0.7.0 <
>> https://github.com/apache/incubator-tuweni/releases/tag/v0.7.0>
>>>>> 
>>>>>> On May 24, 2019, at 05:11, Jim Jagielski <[email protected]> wrote:
>>>>>> 
>>>>>> What tag is this based on?
>>>>>> 
>>>>>>> On May 20, 2019, at 12:39 AM, Antoine Toulme <[email protected]>
>>>>> wrote:
>>>>>>> 
>>>>>>> Hi all,
>>>>>>> 
>>>>>>> We're voting on the distributions available here:
>>>>>>> 
>>>>>>> Specifically, binaries:
>>>>>>> 
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-bin-0.7.0-incubating.tgz
>>>>> <
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-bin-0.7.0-incubating.tgz
>>>>>> 
>>>>>>> 
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-bin-0.7.0-incubating.zip
>>>>> <
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-bin-0.7.0-incubating.zip
>>>>>> 
>>>>>>> Sources:
>>>>>>> 
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-src-0.7.0-incubating.tgz
>>>>> <
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-src-0.7.0-incubating.tgz
>>>>>> 
>>>>>>> 
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-src-0.7.0-incubating.zip
>>>>> <
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-src-0.7.0-incubating.zip
>>>>>> 
>>>>>>> Sample gossip application:
>>>>>>> 
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-gossip-0.7.0-incubating.tgz
>>>>> <
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-gossip-0.7.0-incubating.tgz
>>>>>> 
>>>>>>> 
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-gossip-0.7.0-incubating.zip
>>>>> <
>>>>> 
>> https://dist.apache.org/repos/dist/dev/tuweni/0.7.0/tuweni-gossip-0.7.0-incubating.zip
>>>>>> 
>>>>>>> 
>>>>>>> This is our first release under the Apache incubator. This is the
>>>>> second release candidate.
>>>>>>> 
>>>>>>> Please review and vote. The vote is open until Monday May 27th.
>>>>>>> 
>>>>>>> Cheers,
>>>>>>> 
>>>>>>> Antoine
>>>>>> 
>>>>>> 
>>>>>> ---------------------------------------------------------------------
>>>>>> To unsubscribe, e-mail: [email protected]
>>>>>> For additional commands, e-mail: [email protected]
>>>>>> 
>>>>> 
>>> 
>> 
>> 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to