On Fri, Apr 13, 2018 at 4:03 AM, Jonas Pfefferle <peppe...@japf.ch> wrote:

> Luciano
>
> Thanks for reviewing.
>
> I fixed all the license issues and added rat to the pom file so that you
> can run:
> mvn apache-rat:check
>

Great, thanks


> to verify the build. I excluded all md files. I looked around other Apache
> project and I couldn't find any which put a license header in the md files.
>

You can add license to markdown like this:

<!--
{% comment %}

Add license contents here

{% endcomment %}
-->


> I also fixed the LICENSE file which I believe was copied from Spark and
> had lots of unnecessary licenses in them.
> All these fixes are currently in this pull request:
> https://github.com/apache/incubator-crail/pull/20
> We will merge this soon.
>

Good, please make sure you also remove IBM copyright from all files (and
maybe move to NOTICE file)


>
> Regarding the tests: most of the unit tests cannot be run without a Crail
> config resp. a running Crail namenode/datanode. I propose to disable these
> tests by default and make a build flag to enable them.
> This way people don't have to use skipTests when building Crail.
> What do you guys think?
>

It's documented on the readme that you should run with skipTests, so it
should be ok one way or another.


>
> I'm not sure about the md5 files. I see many Apache and other open source
> projects including them. If you feel strongly about this I will not provide
> them with the next rc.
>

Here is a recent discussion about md5 files
https://www.mail-archive.com/general@incubator.apache.org/msg63139.html


>
> Thanks,
> Jonas
>
>  On Thu, 12 Apr 2018 08:54:05 -0700
>  Luciano Resende <luckbr1...@gmail.com> wrote:
>
>> -1
>>
>> Rat shows many files without a license, please add headers to all files
>> that accept comments such as .java, .pom, .md, etc
>>
>> The LICENSE file, for a source only distribution, should only incorporate
>> what is shipped with the package. The current license seems lime would
>> have
>> been a good one for a binary distribution. In summary, if a package or
>> file
>> is not present in the source distribution, don't list it in the license
>> file.
>>
>> Build runs with skipTest, but mvn clean package fails
>> with java.io.FileNotFoundException: null/conf/crail-site.conf (No such
>> file
>> or directory)
>>
>> Also, looks like recently best practice is to not add .md5 files to
>> distribution, due to its fragilities.
>>
>>
>> On Thu, Apr 12, 2018 at 4:33 AM, Jonas Pfefferle <peppe...@japf.ch>
>> wrote:
>>
>> Hi all,
>>>
>>> I packaged the source and updated the history for our first source
>>> release.
>>>
>>> Thanks to everyone who has contributed to this release.
>>> You can read the release notes here:
>>> https://github.com/apache/incubator-crail/blob/master/HISTORY.md
>>>
>>> The commit to be voted upon:
>>> https://git1-us-west.apache.org/repos/asf?p=incubator-crail.
>>> git;a=commit;h=9f5cfd33316346e159aaaed109abe686762310e0
>>>
>>> Its hash is 9f5cfd33316346e159aaaed109abe686762310e0.
>>>
>>> The artifacts to be voted on are located here:
>>>
>>> https://dist.apache.org/repos/dist/dev/incubator/crail/1.0-rc0/
>>>
>>> The hashes of the artifacts are as follows:
>>> source.tar.gz.md5 28dbb1ae6b15acf2314c97dfd310b438
>>> source.tar.gz.sha256 e8c2b09f2928443c42083f6717c60a
>>> 6190a17ddde8d01dc48ecabd80bfb7d30e
>>> source.zip.md5 8ecef0be5b3363cef6f7fe6a3e053426
>>> source.zip.sha256 b958374bba99c246424153cdd517b0
>>> 4dd8831715b69f62531bf7eaed1aba1055
>>>
>>> Release artifacts are signed with the following key:
>>> https://people.apache.org/keys/committer/pepperjo.asc
>>>
>>> Please vote on releasing this package as Apache Crail 1.0 incubating.
>>>
>>> The vote is open for the next 72 hours and passes if a majority of
>>> at least three +1 PMC votes are cast.
>>>
>>> [ ] +1 Release this package as Apache Crail 1.0 incubating
>>> [ ] 0 I don't feel strongly about it, but I'm okay with the release
>>> [ ] -1 Do not release this package because...
>>> Here is my vote:
>>>
>>> +1
>>>
>>> Jonas
>>>
>>>
>>> --
>> Luciano Resende
>> http://twitter.com/lresende1975
>> http://lresende.blogspot.com/
>>
>
>
>


-- 
Luciano Resende
http://twitter.com/lresende1975
http://lresende.blogspot.com/

Reply via email to