git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] cmake: fix build of `t-oidtree`

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 13, 2024, 01:14 UTC
Message-ID
<xmqq1q3yjdp3.fsf@gitster.g>
In-Reply-To
<D2NXQ2WZFF24.1GVN7WUJ5PCHZ@gmail.com>
"Ghanshyam Thakkar" <shyamthakkar001@gmail.com> writes:
Show 5 quoted lines
> Changes look correct to me. I was not aware of the existance of
> CMakeLists in the tree cause there was no mention of CMake support
> in the docs about building git from source, but it was my mistake
> for grepping for 'test-lib' in just the Makefile instead of project
> wide. Apologies for the negligance and thanks for the patch.

No need to apologize. Nobody caught during the review, so it is not even your fault ;-)

And the CMakeLists is not even meant to work with anything but for Windows, IIRC (even though it might work by accident).

Previous: Ghanshyam ThakkarNext: Johannes Schindelin
Message 5 of 6 in “cmake: fix build of `t-oidtree`”
  1. cmake: fix build of `t-oidtree`Johannes Schindelin via GitGitGadget, Jul 12, 2024
  2. Johannes SchindelinJul 12, 2024
  3. Junio C HamanoJul 12, 2024
  4. Ghanshyam ThakkarJul 12, 2024
  5. Junio C HamanoJul 13, 2024
  6. Johannes SchindelinJul 24, 2024

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.