Thanks Pierre for working on this! We will need the NOTICE file anyways, as the ASF policy[1] says: Every Apache distribution must include a NOTICE file in the top directory, along with the standard LICENSE file.
[1] https://www.apache.org/legal/src-headers.html#notice Yufei On Mon, Sep 22, 2025 at 4:12 AM Robert Stupp <[email protected]> wrote: > Thanks for doing this! > > For the benchmarks/ tool, I guess we need a NOTICE file as well [1], > even if it just contains the "bare minimum", as other (3rd party) work > can be derived from the benchmarks tool. > But even if we do not need it, it doesn't hurt to have it? > > [1] https://infra.apache.org/licensing-howto.html > > On Mon, Sep 22, 2025 at 12:16 PM Pierre Laporte <[email protected]> > wrote: > > > > Hello everyone > > > > The polaris-tools repository does not (yet) conform to the ASF incubation > > guidelines. Typically, tools either do not include a :rat task, the > > LICENSE file, the incubation DISCLAIMER file, or NOTICE files. > > > > I am addressing each problem in a separate pull request so that each PR > is > > trivial to review and does not require too much time. But that will make > > for quite a few PR submitted at once :-). > > > > Also a question for JB: the benchmarks/ tool is not > > packaged/distributed/signed. It can only be run by cloning the > repository > > and running a gradle command. Does it mean that the NOTICE file is not > > necessary there? > > > > -- > > > > Pierre >
