Hi, On Wed, Jun 10, 2026 at 5:04 PM Junwang Zhao <[email protected]> wrote: > > Hi all, > > On Wed, Jun 10, 2026 at 8:56 AM Matt Topol <[email protected]> wrote: > >> > >> > >> > The verify script requirements need to get updated: > >> > > >> > - meson is required but not listed > >> > >> meson is not required to run the verify script, we depend on cmake, they > >> are two independent build systems and meson is not fully supported(e.g, > >> meson doesn't support iceberg-bundle). > > > > > > If meson isn't supposed to be required, then something should change in the > > verify script because cmake failed before it even got to building until I > > installed meson. > > > >> > - gtest/gmock are required but not listed > >> > >> googletest will automatically be made available by cmake's FetchContent, > >> did > >> you run into some problems? > > > > > > Cmake didn't do the fetch content by default then, it failed during > > configure because it couldn't find gtest/gmock. After I installed them, I > > could build. > > > >> > - I had to export CPR_USE_SYSTEM_CURL=OFF for some reason due to ssl > >> > stuff, unsure. > >> > >> I once met an issue that the system curl version is too low(something > >> like curl >= 7.64.0 is required), may I know your > >> installed curl version? We might need to specify the minimum curl > >> version in the README. > > > > > > My system curl is 8.14.1 so this seems odd. I'll try recreating the issue > > tomorrow and capture the exact error. > > It seems there's an issue with how our cpr dependency is being > resolved. Gang has a PR [1] to address it. I'm still trying to > reproduce the Meson/gtest/gmock issue.
Just a heads-up: I was able to run the verify_rc.sh successfully in a freshly created Ubuntu 25.10 container using the following commands: apt update apt-get install -y git curl gpg libcurl4-openssl-dev cmake g++ git clone https://github.com/apache/iceberg-cpp.git cd iceberg-cpp dev/release/verify_rc.sh 0.3.0 3 > > Let's abort this vote. We've uncovered several issues at this early > stage, thanks everyone for the thorough testing and feedback. > > [1] https://github.com/apache/iceberg-cpp/pull/717 > > -- > Regards > Junwang Zhao The issue Matt encountered was likely caused by a bug of our CPR dependency, which has been resolved in RC3. I'll start a new vote thread for RC3. Thanks all. -- Regards Junwang Zhao
