Re: [PATCH] Makefile: Remove excess backslashes from sed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is the same patch as before.  I forgot --dry-run while testing git-send-email.  My apologies.

On Apr 9, 2010, at 10:41 AM, Brian Gernhardt wrote:

> The sed script that was intended to add lines altering the sys.path
> had extra backslashes in them.  Instead resulting in
> 
>  import sys;  import os;  sys.path.insert( ... )
> 
> It output
> 
>  import sys; \ import os; \ sys.path.insert( ... )
> 
> Unfortunately this caused python (2.6.1 on OS X 10.6.3) to error
> 
>  SyntaxError: unexpected character after line continuation character
> 
> Removing two of the backslashes solves this problem.
> 
> Signed-off-by: Brian Gernhardt <brian@xxxxxxxxxxxxxxxxxxxxx>
> ---
> Makefile |    4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Makefile b/Makefile
> index f0fe351..b9eb1ca 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1632,8 +1632,8 @@ $(patsubst %.py,%,$(SCRIPT_PYTHON)): % : %.py
> 	sed -e '1{' \
> 	    -e '	s|#!.*python|#!$(PYTHON_PATH_SQ)|' \
> 	    -e '}' \
> -	    -e 's|^import sys.*|&; \\\
> -	           import os; \\\
> +	    -e 's|^import sys.*|&; \
> +	           import os; \
> 	           sys.path.insert(0, os.getenv("GITPYTHONLIB",\
> 	                                        "@@INSTLIBDIR@@"));|' \
> 	    -e 's|@@INSTLIBDIR@@|'"$$INSTLIBDIR"'|g' \
> -- 
> 1.7.1.rc0.243.g2ce66
> 
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]