Thread (1 message) 1 message, 1 author, 2024-11-20

Re: [PATCH v3 0/4] Build improvements for clar

From: Junio C Hamano <hidden>
Date: 2024-11-20 01:24:13

Justin Tobler [off-list ref] writes:
On 24/11/15 08:32AM, Patrick Steinhardt wrote:
quoted
Hi,

Dscho has reported in [1] that the CMake build instructions for clar do
not work well on Windows/MSVC because we execute the shell scripts
directly instead of using the discovered `SH_EXE`. This small patch
series fixes the issue.

Changes in v2:

    - Wrap overly long lines in the CMake build instructions.
    - Add the VERBATIM option.

Changes in v3:

    - Fix missing word.

Link to v1: https://lore.kernel.org/r/20241108-pks-clar-build-improvements-v1-0-25c1fe65ce37@pks.im (local)
Link to v2: https://lore.kernel.org/r/20241111-pks-clar-build-improvements-v2-0-d4794d8d1b30@pks.im (local)

Thanks!

Patrick
I've reviewed this series and overall this version looks good to me.
Embedding the awk script in the shell script and changing how the
scripts get executed to address platform related build issues seems
sensable to me.
Thanks, both.  Let's merge it down to 'next'.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help