Thanks Fokko for this fix, I've merged this pr. Hi, Kevin:
Would you help to create another rc? On Thu, May 22, 2025 at 5:44 AM Fokko Driesprong <fo...@apache.org> wrote: > Hey everyone, > > I wanted to check the licenses, so I ran the scripts to generate them > since the DEPENDENCIES.rust.tsv files in the repo are empty (an example > here > <https://github.com/apache/iceberg-rust/blob/main/crates/iceberg/DEPENDENCIES.rust.tsv>). > It also includes category B licenses: > > *➜ **iceberg-rust* *git:(**main**) **✗* ./scripts/dependencies.py check > > Checking dependencies of crates/iceberg > > *error[rejected]**: failed to satisfy license requirements* > > ┌─ registry+ > https://github.com/rust-lang/crates.io-index#webpki-roots@0.26.8:4:12 > > │ > > 4 │ license = "MPL-2.0" > > │ ━━━━━━━ > > │ │ > > │ license expression retrieved via Cargo.toml `license` > > │ rejected: license is not explicitly allowed > > │ > > ├ MPL-2.0 - Mozilla Public License 2.0: > > ├ - OSI approved > > ├ - FSF Free/Libre > > ├ - Copyleft > > ├ webpki-roots v0.26.8 > > ├── hyper-rustls v0.27.5 > > │ └── reqwest v0.12.12 > > │ ├── iceberg v0.5.0 > > │ │ └── iceberg-catalog-memory v0.5.0 > > │ │ └── (dev) iceberg v0.5.0 (*) > > │ └── opendal v0.53.0 > > │ └── iceberg v0.5.0 (*) > > └── reqwest v0.12.12 (*) > > > Therefore, I think it would be good to also include the dependency list. > I've created the PR here > <https://github.com/apache/iceberg-rust/pull/1363>. For those that see > the LGPL license in the list, this is for r-efi > <https://github.com/r-efi/r-efi>, which is a triple license, where we can > pick the most favorable one > <https://www.apache.org/legal/resolved.html#mutually-exclusive>. > > > Kind regards, > > Fokko > > Op wo 21 mei 2025 om 19:49 schreef Christian Thiel < > christian.t.b...@gmail.com>: > >> Thanks Kevin! >> +1 non-binding >> >> Checklist for reference: >> [x] Download links are valid. >> [x] Checksums and signatures. >> [x] LICENSE/NOTICE files exist >> [x] No unexpected binary files >> [x] All source files have ASF headers >> [x] Can compile from source >> [x] Doesn't break pipelines badly which are very dear to me >> >> On Wed, 21 May 2025 at 18:32, Zheng Hu <open...@gmail.com> wrote: >> >>> Thank Kevin for the work, Here is my result. >>> >>> +1 binding. >>> >>> Checklist for reference: >>> [x] Download links are valid. >>> [x] Checksums and signatures. >>> [x] LICENSE/NOTICE files exist >>> [x] No unexpected binary files >>> [x] All source files have ASF headers >>> [x] Can compile from source >>> >>> On Wed, May 21, 2025 at 8:37 AM NOTME ZE <st810918...@gmail.com> wrote: >>> >>>> Thank you Kevin for working on this! >>>> >>>> +1 non-binding >>>> >>>> - [x] Download links are valid. >>>> - [x] Checksums and signatures. >>>> - [x] LICENSE/NOTICE files exist >>>> - [x] No unexpected binary files >>>> - [x] All source files have ASF headers >>>> - [x] Can compile from source >>>> >>>> Xuanwo <xua...@apache.org> 于2025年5月21日周三 10:59写道: >>>> >>>>> Thank you Kevin for working on this! >>>>> >>>>> +1 non-binding >>>>> >>>>> - [x] Download links are valid. >>>>> - [x] Checksums and signatures. >>>>> - [x] LICENSE/NOTICE files exist >>>>> - [x] No unexpected binary files >>>>> - [x] All source files have ASF headers >>>>> - [x] Can compile from source >>>>> >>>>> Tested iceberg and iceberg-rest. >>>>> >>>>> On Wed, May 21, 2025, at 01:24, Kevin Liu wrote: >>>>> >>>>> +1 (non-binding) >>>>> >>>>> [x] Download links are valid. >>>>> [x] Checksums and signatures. >>>>> [x] LICENSE/NOTICE files exist >>>>> [x] No unexpected binary files >>>>> [x] All source files have ASF headers >>>>> [x] Can compile from source >>>>> >>>>> Ran `./scripts/verify.py` >>>>> Tested pyiceberg-core against pyiceberg >>>>> >>>>> Best, >>>>> Kevin Liu >>>>> >>>>> On Tue, May 20, 2025 at 8:28 AM Kevin Liu <kevinjq...@apache.org> >>>>> wrote: >>>>> >>>>> Hello Apache Iceberg Rust Community, >>>>> >>>>> This is a call for a vote to release Apache Iceberg rust version 0.5.0. >>>>> The tag to be voted on is v0.5.0-rc.1. >>>>> >>>>> The release candidate: >>>>> >>>>> https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-rust-0.5.0-rc.1/ >>>>> >>>>> Keys to verify the release candidate: >>>>> https://downloads.apache.org/iceberg/KEYS >>>>> >>>>> Git tag for the release: >>>>> https://github.com/apache/iceberg-rust/releases/tag/v0.5.0-rc.1 >>>>> >>>>> Please download, verify, and test. >>>>> >>>>> The VOTE will be open for at least 72 hours and until the necessary >>>>> number of votes are reached. >>>>> [ ] +1 approve >>>>> [ ] +0 no opinion >>>>> [ ] -1 disapprove with the reason >>>>> >>>>> To learn more about Apache Iceberg, please see >>>>> https://rust.iceberg.apache.org/ >>>>> >>>>> Checklist for reference: >>>>> [ ] Download links are valid. >>>>> [ ] Checksums and signatures. >>>>> [ ] LICENSE/NOTICE files exist >>>>> [ ] No unexpected binary files >>>>> [ ] All source files have ASF headers >>>>> [ ] Can compile from source >>>>> >>>>> More detailed checklist please refer to: >>>>> https://github.com/apache/iceberg-rust/tree/main/scripts >>>>> >>>>> To compile from source, please refer to: >>>>> https://github.com/apache/iceberg-rust/blob/main/CONTRIBUTING.md >>>>> >>>>> Here is a Python script in release to help you verify the release >>>>> candidate: >>>>> ./scripts/verify.py >>>>> >>>>> Best, >>>>> Kevin Liu >>>>> >>>>> >>>>> Xuanwo >>>>> >>>>> https://xuanwo.io/ >>>>> >>>>>