Hi Ghanshyam, On Sat, 13 Jul 2024, Ghanshyam Thakkar wrote: > Junio C Hamano <gitster@xxxxxxxxx> wrote: > > "Johannes Schindelin via GitGitGadget" <gitgitgadget@xxxxxxxxx> > > writes: > > > > > From: Johannes Schindelin <johannes.schindelin@xxxxxx> > > > > > > When the `oidtree` test helper was turned into a unit test, a new > > > `lib-oid` source file was added as dependency. This was only done in the > > > Makefile so far, but also needs to be done in the CMake definition. > > > > > > This is a companion of ed548408723d (t/: migrate helper/test-oidtree.c > > > to unit-tests/t-oidtree.c, 2024-06-08). > > > > > > Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx> > > > --- > > > cmake: fix build of t-oidtree > > > > > > This is based on the gt/unit-test-oidtree branch. > > > > > > /cc "Ghanshyam Thakkar" shyamthakkar001@xxxxxxxxx > > > > The base topic has been merged last month during this cycle, so the > > fix must be merged down to the 'master' before the release. Will > > queue. > > Changes look correct to me. Thank you for confirming. For what it's worth, it was this confirmation by the domain expert (i.e. you) that I was looking for when I Cc:ed you, not to spread blame. The CMakeLists.txt file exists primarily to support building Git in Visual Studio, and I removed that part from CI builds specifically so that contributors on the Git mailing list need not worry about that support. Ciao, Johannes
