# [PATCH V4] git on Mac OS and precomposed unicode

2 messages from 2012-01-21 to 2012-01-21. Participants: Torsten Bögershausen, Junio C Hamano.
Thread: https://gitlist.dev/t/29421

## Torsten Bögershausen, 2012-01-21 19:36

Subject: [PATCH V4] git on Mac OS and precomposed unicode
Message-ID: <201201212036.46681.tboegi@web.de>
URL: https://gitlist.dev/e/201201212036.46681.tboegi%40web.de

```
Changes since V3:

- Be much more defensive:
  Do not set core.precomposedunicode=true, even if the file system
  probing indicates that it could be true.
  (but we don't say that it should be true).
  This is to keep better backward compatibility within git.
  The user needs to manually enable the precomposition.
  However, the .git/config indicates that we have a new
  configuration variable, and will encourage people to use it.

- compat/precomposed_utf8.c: 
    - re-order #includes
    - Added some empty lines to make code easier to read. 
    - Small fixes (xmalloc, errno handling)

- Make function names more consistent by renaming these functions:
    argv_precompose() -> precompose_argv()
    str_precompose() -> precompose_str()
- Improved the commit message


Torsten Bögershausen (1):
  git on Mac OS and precomposed unicode

 Documentation/config.txt     |    9 ++
 Makefile                     |    3 +
 builtin/init-db.c            |    2 +
 compat/precomposed_utf8.c    |  208 ++++++++++++++++++++++++++++++++++++++++++
 compat/precomposed_utf8.h    |   30 ++++++
 git-compat-util.h            |    9 ++
 git.c                        |    1 +
 t/t3910-mac-os-precompose.sh |  117 +++++++++++++++++++++++
 8 files changed, 379 insertions(+), 0 deletions(-)
 create mode 100644 compat/precomposed_utf8.c
 create mode 100644 compat/precomposed_utf8.h
 create mode 100755 t/t3910-mac-os-precompose.sh

-- 
1.7.8.rc0.43.gb49a8

```

## Junio C Hamano, 2012-01-21 22:14

Subject: Re: [PATCH V4] git on Mac OS and precomposed unicode
Message-ID: <7vwr8kaejy.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vwr8kaejy.fsf%40alter.siamese.dyndns.org
In-Reply-To: <201201212036.46681.tboegi@web.de>

```
Torsten Bögershausen <tboegi@web.de> writes:

> Changes since V3:
>
> - Be much more defensive:
>   Do not set core.precomposedunicode=true, even if the file system
>   probing indicates that it could be true.
>   (but we don't say that it should be true).
>   This is to keep better backward compatibility within git.
>   The user needs to manually enable the precomposition.
>   However, the .git/config indicates that we have a new
>   configuration variable, and will encourage people to use it.
>
> - compat/precomposed_utf8.c: 
>     - re-order #includes
>     - Added some empty lines to make code easier to read. 
>     - Small fixes (xmalloc, errno handling)
>
> - Make function names more consistent by renaming these functions:
>     argv_precompose() -> precompose_argv()
>     str_precompose() -> precompose_str()
> - Improved the commit message

Next time around, please place these after --- lines in the patch message
itself, without making a single-patch series with a separate cover letter.

Thanks.

```