The update hook template intends to abort if the project description file hasn't been adjusted or is empty. This patch fixes the check for 'being adjusted'. Signed-off-by: Gerrit Pape <pape@xxxxxxxxxxx> --- On Tue, Nov 06, 2007 at 02:55:55PM +0100, Andreas Ericsson wrote: > Gerrit Pape wrote: > > # check for no description > >-projectdesc=$(sed -e '1p' "$GIT_DIR/description") > >-if [ -z "$projectdesc" -o "$projectdesc" = "Unnamed repository; edit > >this > >file to name it for gitweb" ]; then > >+projectdesc=$(sed -ne '1p' "$GIT_DIR/description") > > Write this as > projectdesc=$(sed -e 1q "$GIT_DIR/description") > instead. It's a little shorter, a little faster and slightly more > portable. I made it $(head -n 1 ...). templates/hooks--update | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/templates/hooks--update b/templates/hooks--update index d8c7626..fbf664f 100644 --- a/templates/hooks--update +++ b/templates/hooks--update @@ -34,8 +34,8 @@ fi allowunannotated=$(git-repo-config --bool hooks.allowunannotated) # check for no description -projectdesc=$(sed -e '1p' "$GIT_DIR/description") -if [ -z "$projectdesc" -o "$projectdesc" = "Unnamed repository; edit this file to name it for gitweb" ]; then +projectdesc=$(head -n 1 "$GIT_DIR/description") +if [ -z "$projectdesc" -o "$projectdesc" = "Unnamed repository; edit this file to name it for gitweb." ]; then echo "*** Project description file hasn't been set" >&2 exit 1 fi -- 1.5.3.5 - 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
