filter-branch: use $SHELL_PATH instead of 'sh'
On some systems, 'sh' isn't very friendly. In particular, t7003 fails on Solaris because it doesn't understand $(). Instead, use the specified SHELL_PATH to run shell code. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>maint
							parent
							
								
									d89c1dfac9
								
							
						
					
					
						commit
						4bf9f27dfb
					
				
							
								
								
									
										1
									
								
								Makefile
								
								
								
								
							
							
						
						
									
										1
									
								
								Makefile
								
								
								
								
							|  | @ -873,6 +873,7 @@ common-cmds.h: $(wildcard Documentation/git-*.txt) | ||||||
| $(patsubst %.sh,%,$(SCRIPT_SH)) : % : %.sh | $(patsubst %.sh,%,$(SCRIPT_SH)) : % : %.sh | ||||||
| 	$(QUIET_GEN)$(RM) $@ $@+ && \ | 	$(QUIET_GEN)$(RM) $@ $@+ && \ | ||||||
| 	sed -e '1s|#!.*/sh|#!$(SHELL_PATH_SQ)|' \ | 	sed -e '1s|#!.*/sh|#!$(SHELL_PATH_SQ)|' \ | ||||||
|  | 	    -e 's|@SHELL_PATH@|$(SHELL_PATH_SQ)|' \ | ||||||
| 	    -e 's|@@PERL@@|$(PERL_PATH_SQ)|g' \ | 	    -e 's|@@PERL@@|$(PERL_PATH_SQ)|g' \ | ||||||
| 	    -e 's/@@GIT_VERSION@@/$(GIT_VERSION)/g' \ | 	    -e 's/@@GIT_VERSION@@/$(GIT_VERSION)/g' \ | ||||||
| 	    -e 's/@@NO_CURL@@/$(NO_CURL)/g' \ | 	    -e 's/@@NO_CURL@@/$(NO_CURL)/g' \ | ||||||
|  |  | ||||||
|  | @ -309,7 +309,7 @@ while read commit parents; do | ||||||
| 	sed -e '1,/^$/d' <../commit | \ | 	sed -e '1,/^$/d' <../commit | \ | ||||||
| 		eval "$filter_msg" > ../message || | 		eval "$filter_msg" > ../message || | ||||||
| 			die "msg filter failed: $filter_msg" | 			die "msg filter failed: $filter_msg" | ||||||
| 	sh -c "$filter_commit" "git commit-tree" \ | 	@SHELL_PATH@ -c "$filter_commit" "git commit-tree" \ | ||||||
| 		$(git write-tree) $parentstr < ../message > ../map/$commit | 		$(git write-tree) $parentstr < ../message > ../map/$commit | ||||||
| done <../revs | done <../revs | ||||||
|  |  | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue
	
	 Jeff King
						Jeff King