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

Re: [PATCH/RFC] Auto detection in Makefile if msgfmt is not available

From
Torsten Bögershausen <tboegi@web.de>
Date
Feb 12, 2012, 19:09 UTC
Message-ID
<DA8341B0-2A4F-40D8-A5CB-F8159E087C98@web.de>
In-Reply-To
<7vr4y0j79w.fsf@alter.siamese.dyndns.org>
Am 12.02.2012 um 18:22 schrieb Junio C Hamano:
Show 30 quoted lines
> Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
> 
>> 2012/2/12 Torsten Bögershausen <tboegi@web.de>:
>>> Added a simple auto-detection and switch to NO_GETTEXT when
>>> msgfmt could not be found on the system
>> 
>> Oh look, a start at our very own autoconf replacement :)
> 
> It is a bad idea for the Makefile to silently decide to flip NO_GETTEXT
> when the system happens to be missing msgfmt without letting the user know
> what is happening.  Current behaviour to error out will at least give an
> opportunity to stop and think if installing gettext suite on the system
> makes sense before proceeding.
> 
> Given that the Makefile only has this to say:
> 
>    # Define NO_GETTEXT if you don't want Git output to be translated.
>    # A translated Git requires GNU libintl or another gettext implementation,
>    # plus libintl-perl at runtime.
> 
> expecting that "msgfmt: no such command" clicks "Ah, I do not have gettext
> suite" for anybody who attempts to build (and fail) Git, it however is
> also a bit unfair and unhelpful.
> 
> Perhaps something like this is necessary and sufficient.
> 
>    # Define NO_GETTEXT if you don't want Git output to be translated or if
>    # you do not have gettext suite (e.g. "msgfmt" and "gettext" commands).
>    # A translated Git requires GNU libintl or another gettext implementation,
>    # plus libintl-perl at runtime.

Thanks all! a) the problem was fixed by "sudo apt-get install gettext", which leads to b) The comment was very helpful ( As I didn't know which package to install) and the final question: c) Should I send a new patch ? /Torsten

Previous: Junio C Hamano
Message 4 of 4 in “Auto detection in Makefile if msgfmt is not available”
  1. Auto detection in Makefile if msgfmt is not availableTorsten Bögershausen, Feb 12, 2012
  2. Ævar Arnfjörð BjarmasonFeb 12, 2012
  3. Junio C HamanoFeb 12, 2012
  4. Torsten BögershausenFeb 12, 2012

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.