*bq. no need to add anything into NOTICE*
Correction: here we need to analyze the Spark NOTICE file and the relevant
portions bubbled up into the top-level NOTICE file [1], as already pointed
out by Justin in previous reply [2].

Best Regards,
Yu

[1] https://infra.apache.org/licensing-howto.html#alv2-dep
[2] https://lists.apache.org/thread/bopd2yql1nvjt1kmn9vcd86zqg924g21

On Tue, 31 Jan 2023 at 14:03, Yu Li <car...@gmail.com> wrote:

> Thanks for the further clarification Justin, it helps a lot. And thanks
> for correcting me. Now I understand that I should have referred to the
> 3rd-party section [1] instead.
>
> Explicitly, I think the action-to-do is to add (append) the below lines
> into the Celeborn LICENSE file [2] (similar to that of Spark [3]), meantime
> keep the copyright text in the source file and no need to add anything into
> NOTICE:
>
>
> *------------------------------------------------------------------------------------*
>
> *This product bundles various third-party components under other open
> source licenses.*
>
> *This section summarizes those components and their licenses. See
> licenses/*
>
> *for text of these licenses.*
>
>
>
> *Apache Software Foundation License 2.0*
>
> *--------------------------------------*
>
>
>
> *common/src/main/java/org/apache/celeborn/common/network/util/LimitedInputStream.java*
>
>
>
> ======================
> PS. attaching the detailed license instructions below for easier reference:
>
>    1. The term "third-party work" refers to a work not submitted directly
>    to the ASF by the copyright owner or owner's agent. This includes parts of
>    a work submitted directly to the ASF for which the submitter is not the
>    copyright owner or owner's agent.
>    2. Do not modify or remove any copyright notices or licenses within
>    third-party works.
>    3. Make sure that every third-party work includes its associated
>    license, even if that requires adding a copy of the license from the
>    third-party download site into the distribution.
>    4. Do not add the standard Apache License header to the top of
>    third-party source files.
>    5. Minor modifications/additions to third-party source files should
>    typically be licensed under the same terms as the rest of the third-party
>    source for convenience.
>    6. The project's PMC should deal with major modifications/additions to
>    third-party source files on a case-by-case basis.
>
>
> Best Regards,
> Yu
>
> [1] https://www.apache.org/legal/src-headers.html#3party
> [2] https://github.com/apache/incubator-celeborn/blob/main/LICENSE
> [3] https://github.com/apache/spark/blob/master/LICENSE#L204-L213
>
>
> On Tue, 31 Jan 2023 at 13:11, Justin Mclean <jus...@classsoftware.com>
> wrote:
>
>> Hi,
>>
>> > *bq. It belongs in LICENSE not NOTICE only copyright notices that have
>> been
>> > removed (by or with the permission of the owner) get listed in NOTICE*
>> > Since Guava is also with ALv2, referring to [1], my understanding is
>> that:
>> > 1. There should be no copyright notice in the header (#2)
>> > 2. We need to move the copyright to the NOTICE file associated with each
>> > applicable project release (#1.b)
>> >
>> > And referring to [2], "Copyright notifications which have been
>> relocated,
>> > rather than removed, from source files must be preserved in NOTICE
>> (unless
>> > embedded in MIT/BSD license)". I'm not sure whether reserving the
>> copyright
>> > in Spark way means "relocate", but if it is, then it seems the copyright
>> > should occur (be reserved) in NOTICE.
>>
>> Sorry that's not correct as:
>> - Relocated in this context means that the owner has given the software
>> to the ASF (usually via a software grant).
>> - Guava is 3rd party software so it needs to be mentioned in LICENSE.
>> - Guava is not an ASF project so its header would have a copyright line
>>
>> This will probably help [1]
>>
>> Kind Regards,
>> Justin
>>
>> 1. https://infra.apache.org/licensing-howto.html#alv2-dep
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
>> For additional commands, e-mail: general-h...@incubator.apache.org
>>
>>

Reply via email to