tuhaihe opened a new pull request, #154: URL: https://github.com/apache/cloudberry-pxf/pull/154
## What does this PR do? Removes two license-pointer paragraphs from `NOTICE`, keeping the attributions it is actually there to carry. ## Why During the [2.2.0-rc1 IPMC vote](https://lists.apache.org/thread/yyz57l860ktsx7txqyqrdkomz40pobcw), Justin Mclean listed under "should fix" that the NOTICE files of the cloudberry-backup and cloudberry-go-libs components carry Apache license boilerplate, while NOTICE is reserved for notifications that are legally required. He did not review this repository, but the same issue is here in a milder form, so this PR is the counterpart of the fixes in those two. The text is inherited rather than invented: `greenplum-db/pxf-archive` ships a NOTICE whose body is the VMware attribution followed by these two paragraphs, and both were copied in when the code was donated. Section 4(d) of the Apache License requires "the attribution notices contained within such NOTICE file" — the notices, not the file. And [infra.apache.org/licensing-howto.html](https://infra.apache.org/licensing-howto.html) is explicit: > its contents must be analyzed and the **relevant portions** bubbled up into > the top-level NOTICE file > It is important to keep NOTICE as brief and simple as possible, as each > addition places a burden on downstream consumers. **Do not add anything to > NOTICE which is not legally required.** ## What goes ``` This product is licensed to you under the Apache 2.0 license (the "License"). You may not use this product except in compliance with the Apache 2.0 License. This product may include a number of subcomponents with separate copyright notices and license terms. Your use of these subcomponents is subject to the terms and conditions of the subcomponent's license, as noted in the LICENSE file. ``` Neither is an attribution notice. The first restates the license, which `LICENSE` already conveys; the second points at `LICENSE` for subcomponent terms, which is where a reader would look anyway. The Apache HAWQ and VMware attributions are untouched. ## Type of Change - [x] Documentation update ## Impact **User-facing changes:** `NOTICE` is shorter. No functional change. ## Checklist - [x] Followed [contribution guide](https://cloudberry.apache.org/contribute/code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
