[PATCH] D96482: [DeveloperPolicy] Specify LLVM's license more clearly.
Thorsten via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 11 04:28:59 PST 2021
tschuett added a comment.
In D96482#2556325 <https://reviews.llvm.org/D96482#2556325>, @kristof.beyls wrote:
> In D96482#2556315 <https://reviews.llvm.org/D96482#2556315>, @tschuett wrote:
>
>> If you would extend the paragraph, you could say:
>> "For reference the SPDX identifier is ..."
>
> Yes, it could be helpful to also explicitly call out the SPDX identifier. However, there is also value in keeping that introductory paragraph as short as possible.
> On balance, I think it's slightly better to not call out the SPDX identifier in that introductory paragraph.
> Maybe there's value in explicitly calling out the SPDX identifiers in one of the later paragraphs?
Agreed. In fact, I could neither find a WITH nor a SPDX in the whole file.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96482/new/
https://reviews.llvm.org/D96482
More information about the llvm-commits
mailing list