Re: [PATCH] INSTALL: add macOS gettext and sdk header explanation to INSTALL

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Nov 15, 2018, 03:05 UTC
Message-ID: <20181115030503.GC92137@google.com>
In-Reply-To: <20181115013233.24685-1-yanke131415@gmail.com>

Hi!

yanke131415@gmail.com wrote:

Show 5 quoted lines

> From: out0fmemory <jiu4majia2@163.com>
> 
> ---
>  INSTALL | 7 +++++++
>  1 file changed, 7 insertions(+)

Thanks for writing. A few bits of administrivia, from https://www.kernel.org/pub/software/scm/git/docs/SubmittingPatches.html:

Can we forge your sign-off? See the section "certify your work" in SubmittingPatches for what this means.

What name should we attribute this change to? Documentation/SubmittingPatches explains:

 Also notice that a real name is used in the Signed-off-by: line.
 Please don’t hide your real name.

[...]

Show 9 quoted lines

> --- a/INSTALL
> +++ b/INSTALL
> @@ -165,6 +165,9 @@ Issues of note:
>  	  use English. Under autoconf the configure script will do this
>  	  automatically if it can't find libintl on the system.
>  
> +    In macOS, can install and link gettext with brew as "brew install gettext"
> +    and "brew link --force gettext", the link operation is necessary.
> +

As context (e.g. to go in the commit message), can you tell us more about this? E.g. what happens if you don't perform the link operation?

[...]

Show 8 quoted lines

> @@ -178,6 +181,10 @@ Issues of note:
>     will include them.  Note that config.mak is not distributed;
>     the name is reserved for local settings.
>  
> +  - In macOs 10.14, the Command Line Tool not contains the headers as before, so it
> +    need install Command Line Tool 10.14 and install the headers which command
> +    "sudo installer -pkg /Library/Developer/CommandLineTools/Packages/macOS_SDK_headers_for_macOS_10.14.pkg -target".
> +

Likewise: what is the symptom if this isn't done?

Is there more general advice we can put here that will survive past macOS 10.14?

Thanks and hope that helps, Jonathan