Re: [PATCH 1/3] git-bisect: war on "sed"

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Wincent Colaiuta
Date: Thursday, November 15, 2007 - 2:36 am

El 15/11/2007, a las 10:29, Miles Bader escribió:


Nothing, but using it means forking a new process unnecessarily, and  
the shorter form without sed is arguably more readable:

-		echo "$head" | sed 's#^refs/heads/##' >"$GIT_DIR/head-name"
+		echo "${head#refs/heads/}" >"$GIT_DIR/head-name"

Cheers,
Wincent



-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH v2] Bisect reset: remove bisect refs that may have ..., Christian Couder, (Thu Nov 15, 12:18 am)
[PATCH 1/3] git-bisect: war on "sed", Junio C Hamano, (Thu Nov 15, 2:00 am)
[PATCH 3/3] git-bisect: modernize branch shuffling hack, Junio C Hamano, (Thu Nov 15, 2:01 am)
Re: [PATCH 1/3] git-bisect: war on "sed", Miles Bader, (Thu Nov 15, 2:29 am)
Re: [PATCH 1/3] git-bisect: war on "sed", Wincent Colaiuta, (Thu Nov 15, 2:36 am)
Re: [PATCH 1/3] git-bisect: war on "sed", Miles Bader, (Thu Nov 15, 2:53 am)
Re: [PATCH 1/3] git-bisect: war on "sed", Andreas Ericsson, (Thu Nov 15, 3:06 am)
Re: [PATCH 1/3] git-bisect: war on "sed", David Kastrup, (Thu Nov 15, 3:14 am)
Re: [PATCH v2] Bisect reset: remove bisect refs that may h ..., Johannes Schindelin, (Thu Nov 15, 5:10 am)