Re: [PATCH v3 11/11] cmake: fix typo in message when `msgfmt` was not found
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 28, 2020, 22:11 UTC
- Message-ID
- <xmqqtuvhk0vy.fsf@gitster.c.googlers.com>
- In-Reply-To
- <c3e235fe39b41b90a8a00d81220cb28e83580f13.1601327357.git.gitgitgadget@gmail.com>
"Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com> writes:
> From: Johannes Schindelin <johannes.schindelin@gmx.de> > > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> > ---
Shouldn't this be squashed into the step being fixed, which is part of this series? Would it be the "fall back to using..." step (4/11)?
Show 16 quoted lines
> contrib/buildsystems/CMakeLists.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/contrib/buildsystems/CMakeLists.txt b/contrib/buildsystems/CMakeLists.txt
> index 4b16744847..b9fced682e 100644
> --- a/contrib/buildsystems/CMakeLists.txt
> +++ b/contrib/buildsystems/CMakeLists.txt
> @@ -166,7 +166,7 @@ find_program(MSGFMT_EXE msgfmt)
> if(NOT MSGFMT_EXE)
> set(MSGFMT_EXE ${CMAKE_SOURCE_DIR}/compat/vcbuild/vcpkg/downloads/tools/msys2/msys64/usr/bin/msgfmt.exe)
> if(NOT EXISTS ${MSGFMT_EXE})
> - message(WARNING "Text Translations won't be build")
> + message(WARNING "Text Translations won't be built")
> unset(MSGFMT_EXE)
> endif()
> endif()