I don't think there's a license issue. Looks like there's already a RAT check in the build steps. https://github.com/apache/polaris/blob/e46c6cbb61e69dcb12775fa262c09437f8ee8a28/build.gradle.kts#L59-L124 And the files listed above are excluded.
Best, Kevin Liu On Mon, Nov 18, 2024 at 3:03 PM Russell Spitzer <russell.spit...@gmail.com> wrote: > Shouldn't we fix the license issues before the release? Seems like an > important and easy thing to do. I also think we should be excluding the > "site" directory from the release? > > > On Mon, Nov 18, 2024 at 4:56 PM Dmitri Bourlatchkov > <dmitri.bourlatch...@dremio.com.invalid> wrote: > > > +1 (non-binding) > > > > Validated sha512 and signature. > > > > Cheers, > > Dmitri. > > > > On Mon, Nov 18, 2024 at 3:16 PM Jean-Baptiste Onofré <j...@nanthrax.net> > > wrote: > > > > > Hi Dmitri > > > > > > I updated dist with the correct sha512. > > > > > > I noticed several gradle issues during the release (in signature, > > > maven publication, etc). I will create the issues/PRs to fix that with > > > the release process documentation. > > > > > > Regards > > > JB > > > > > > On Mon, Nov 18, 2024 at 3:03 PM Dmitri Bourlatchkov > > > <dmitri.bourlatch...@dremio.com.invalid> wrote: > > > > > > > > I noticed that the apache-polaris-0.9.0-incubating.sha512 file > contains > > > the > > > > right hash, but it's format does not match the output of `sha512sum > > > > apache-polaris-0.9.0-incubating.tar.gz` -- files names are not listed > > > after > > > > hashes... > > > > > > > > As a result, sha512sum -c apache-polaris-0.9.0-incubating.sha512 > > returns > > > a > > > > formal error. > > > > > > > > Are we ok with that? > > > > > > > > For reference, the httpd hashes do list file names: > > > > https://downloads.apache.org/httpd/httpd-2.4.62.tar.bz2.sha512 > > > > > > > > Cheers, > > > > Dmitri. > > > > > > > > On Sun, Nov 17, 2024 at 1:00 AM Jean-Baptiste Onofré < > j...@nanthrax.net> > > > > wrote: > > > > > > > > > Hi everyone, > > > > > > > > > > I propose that we release the following RC as the official Apache > > > > > Polaris 0.9.0-incubating release. > > > > > > > > > > * This corresponds to the tag: apache-polaris-0.9.0-incubating-rc1 > > > > > * > > > > > > > > > > > https://github.com/apache/polaris/commits/apache-polaris-0.9.0-incubating-rc1 > > > > > * > > > > > > > > > > > https://github.com/apache/polaris/tree/445c42768d1e3148c912ac5c45ee53036b9ef318 > > > > > > > > > > The release tarball, signature, and checksums are here: > > > > > * > > > > > > > > > > > https://dist.apache.org/repos/dist/dev/incubator/polaris/0.9.0-incubating/ > > > > > > > > > > You can find the KEYS file here: > > > > > * > https://dist.apache.org/repos/dist/release/incubator/polaris/KEYS > > > > > > > > > > Convenience binary artifacts are staged on Nexus. The Maven > > repository > > > URL > > > > > is: > > > > > * > > > > > > > > > > > https://repository.apache.org/content/repositories/orgapachepolaris-1003/ > > > > > > > > > > Please download, verify, and test. > > > > > > > > > > Please vote in the next 72 hours. > > > > > [ ] +1 Release this as Apache polaris 0.9.0-incubating > > > > > [ ] +0 > > > > > [ ] -1 Do not release this because... > > > > > > > > > > Only PPMC members and mentors have binding votes, but other > community > > > > > members are > > > > > encouraged to cast non-binding votes. This vote will pass if there > > are > > > > > 3 binding +1 votes and more binding +1 votes than -1 votes. > > > > > > > > > > NB: if this vote passes, a new vote will be started on the > Incubator > > > > > general mailing > > > > > list. > > > > > > > > > > Thanks > > > > > Regards > > > > > JB > > > > > > > > > > >
