threads / patch / 40228

patchINSTALL: Add docs on how to use custom Python paths.

Subject: [PATCH] INSTALL: Add docs on how to use custom Python paths.

## tl;dr

3 messages between Aug 31, 2015 and Aug 31, 2015. Diffs are folded; open one to read it.

replies: 2people: 3as markdown or json

Ehsan Akhgari· Aug 31, 2015, 13:22 UTC · lore

Signed-off-by: Ehsan Akhgari <ehsan@mozilla.com>
---
 INSTALL | 5 +++++
 1 file changed, 5 insertions(+)

Show changes to INSTALL +5 −0

diff --git a/INSTALL b/INSTALL
index ffb071e..5b7fe91 100644
--- a/INSTALL
+++ b/INSTALL
@@ -148,6 +148,11 @@ Issues of note:
 	  supported by Perforce) is needed to use the git-p4 interface
 	  to Perforce.
 
+	  By default, git looks for Python installed in /usr/bin/python.
+	  On systems with a different Python installation path, pass the
+	  --with-python argument to configure, or set the PYTHON_PATH
+	  environment variable when using make to build.
+
  - Some platform specific issues are dealt with Makefile rules,
    but depending on your specific installation, you may not
    have all the libraries/tools needed, or you may have

-- 
2.4.0

Luke Diamand· Aug 31, 2015, 14:28 UTC · re: Ehsan Akhgari · lore

Re: [PATCH] INSTALL: Add docs on how to use custom Python paths.

On 31 August 2015 at 14:22, Ehsan Akhgari <ehsan@mozilla.com> wrote:

Show 21 quoted lines

> Signed-off-by: Ehsan Akhgari <ehsan@mozilla.com>
> ---
>  INSTALL | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/INSTALL b/INSTALL
> index ffb071e..5b7fe91 100644
> --- a/INSTALL
> +++ b/INSTALL
> @@ -148,6 +148,11 @@ Issues of note:
>           supported by Perforce) is needed to use the git-p4 interface
>           to Perforce.
>
> +         By default, git looks for Python installed in /usr/bin/python.
> +         On systems with a different Python installation path, pass the
> +         --with-python argument to configure, or set the PYTHON_PATH
> +         environment variable when using make to build.
> +
>   - Some platform specific issues are dealt with Makefile rules,
>     but depending on your specific installation, you may not
>     have all the libraries/tools needed, or you may have

Looks good to me.

Thanks Luke

Junio C Hamano· Aug 31, 2015, 14:53 UTC · re: Ehsan Akhgari · lore

Re: [PATCH] INSTALL: Add docs on how to use custom Python paths.

Ehsan Akhgari <ehsan@mozilla.com> writes:

Show 21 quoted lines

> Signed-off-by: Ehsan Akhgari <ehsan@mozilla.com>
> ---
>  INSTALL | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/INSTALL b/INSTALL
> index ffb071e..5b7fe91 100644
> --- a/INSTALL
> +++ b/INSTALL
> @@ -148,6 +148,11 @@ Issues of note:
>  	  supported by Perforce) is needed to use the git-p4 interface
>  	  to Perforce.
>  
> +	  By default, git looks for Python installed in /usr/bin/python.
> +	  On systems with a different Python installation path, pass the
> +	  --with-python argument to configure, or set the PYTHON_PATH
> +	  environment variable when using make to build.
> +
>   - Some platform specific issues are dealt with Makefile rules,
>     but depending on your specific installation, you may not
>     have all the libraries/tools needed, or you may have

Thanks for a patch, but I doubt that it is a good idea to do this.

The paragraph we see in the post-context of your patch is meant to cover it, together with 47 other platform specific locations like SHELL_PATH, PERL_PATH, CURLDIR, etc.