incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joshua Poore <poor...@me.com.INVALID>
Subject Re: [VOTE] Release Apache Doris 0.10.0-incubating-rc04
Date Sun, 30 Jun 2019 04:13:48 GMT
Hi Justin,

Thanks for the review!

RE License appendix:

Is it just the Copyright symbol that’s incorrect? Or do you see something else. Just checking
so that I know what to fix. Its the only thing I can see as different from [1]

reads as follwos:

   APPENDIX: How to apply the Apache License to your work.

      To apply the Apache License to your work, attach the following
      boilerplate notice, with the fields enclosed by brackets "[]"
      replaced with your own identifying information. (Don't include
      the brackets!)  The text should be enclosed in the appropriate
      comment syntax for the file format. We also recommend that a
      file or class name and description of purpose be included on the
      same "printed page" as the copyright notice for easier
      identification within third-party archives.

   © Copyright [yyyy] [name of copyright owner]

   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
   You may obtain a copy of the License at

       http://www.apache.org/licenses/LICENSE-2.0

   Unless required by applicable law or agreed to in writing, software
   distributed under the License is distributed on an "AS IS" BASIS,
   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   See the License for the specific language governing permissions and
   limitations under the License.

[1] https://www.apache.org/licenses/LICENSE-2.0 <https://www.apache.org/licenses/LICENSE-2.0>

> On Jun 29, 2019, at 8:53 PM, Justin Mclean <justin@classsoftware.com> wrote:
> 
> Hi,
> 
> +1 (binding)
> 
> I checked:
> - incubating in name
> - DISCLAIMER exists
> - LICENSE has some minor issues (see below)
> - NOTICE is fine
> - ASF file have ASF headers
> - No unexpected binary files
> - I didn’t compile as I’m on an unsupported platform
> 
> LICENSE is missing:
> - UTF-8 library [1][2]
> - These files [3][4] copyright Facebook. The headers in this files may not be correct.
> - MIT licensed Sizzle from [6]
> 
> The licensing information list in headers of [3][4] is confusing e.g Is it under an APLv2
license, or an AL or GPL license, or a BSD license? You might also want to read this JIRA
[5] to double check that the version of rocksDB you are using is OK and not Category X.
> 
> Thanks,
> Justin
> 
> 1. ./be/src/gutil/utf/LICENSE
> 2. ./be/src/gutil/utf/rune.c
> 3. ./be/src/util/coding.cpp
> 4. ./be/src/util/coding.cpp
> 5. https://issues.apache.org/jira/browse/LEGAL-303
> 6. /webroot/static/jquery.js
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
> 


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message