# [PATCH] post-receive-email: suppress error if description file missing

3 messages from 2011-02-02 to 2011-02-03. Participants: Sitaram Chamarty, Junio C Hamano.
Thread: https://gitlist.dev/t/26390

## Sitaram Chamarty, 2011-02-02 17:42

Subject: [PATCH] post-receive-email: suppress error if description file missing
Message-ID: <20110202174242.GA12470@atcmail.atc.tcs.com>
URL: https://gitlist.dev/e/20110202174242.GA12470%40atcmail.atc.tcs.com

```
---
 contrib/hooks/post-receive-email |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email
index f99ea95..71f2b47 100755
--- a/contrib/hooks/post-receive-email
+++ b/contrib/hooks/post-receive-email
@@ -709,7 +709,7 @@ if [ -z "$GIT_DIR" ]; then
 	exit 1
 fi
 
-projectdesc=$(sed -ne '1p' "$GIT_DIR/description")
+projectdesc=$(sed -ne '1p' "$GIT_DIR/description 2>/dev/null")
 # Check if the description is unchanged from it's default, and shorten it to
 # a more manageable length if it is
 if expr "$projectdesc" : "Unnamed repository.*$" >/dev/null
-- 
1.7.3.4

```

## Junio C Hamano, 2011-02-02 18:36

Subject: Re: [PATCH] post-receive-email: suppress error if description file missing
Message-ID: <7vbp2uxozo.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vbp2uxozo.fsf%40alter.siamese.dyndns.org
In-Reply-To: <20110202174242.GA12470@atcmail.atc.tcs.com>

```
Sitaram Chamarty <sitaramc@gmail.com> writes:

> ---
>  contrib/hooks/post-receive-email |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email
> index f99ea95..71f2b47 100755
> --- a/contrib/hooks/post-receive-email
> +++ b/contrib/hooks/post-receive-email
> @@ -709,7 +709,7 @@ if [ -z "$GIT_DIR" ]; then
>  	exit 1
>  fi
>  
> -projectdesc=$(sed -ne '1p' "$GIT_DIR/description")
> +projectdesc=$(sed -ne '1p' "$GIT_DIR/description 2>/dev/null")

I suspect that you would want to have the redirection outside the dq pair,
but other than that, well spotted.

You forgot to sign-off, though ;-)

>  # Check if the description is unchanged from it's default, and shorten it to
>  # a more manageable length if it is
>  if expr "$projectdesc" : "Unnamed repository.*$" >/dev/null
> -- 
> 1.7.3.4

```

## Sitaram Chamarty, 2011-02-03 01:00

Subject: Re: [PATCH] post-receive-email: suppress error if description file missing
Message-ID: <20110203010032.GA30329@atcmail.atc.tcs.com>
URL: https://gitlist.dev/e/20110203010032.GA30329%40atcmail.atc.tcs.com
In-Reply-To: <7vbp2uxozo.fsf@alter.siamese.dyndns.org>

```
Signed-off-by: Sitaram Chamarty <sitaramc@gmail.com>
---

On Wed, Feb 02, 2011 at 10:36:43AM -0800, Junio C Hamano wrote:

> I suspect that you would want to have the redirection outside the dq pair,
> but other than that, well spotted.

doh!  Thanks for catching the stupid error -- I shouldn't attempt even a
simple patch when half-asleep I guess :)

> You forgot to sign-off, though ;-)

done!

 contrib/hooks/post-receive-email |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email
index f99ea95..21989fc 100755
--- a/contrib/hooks/post-receive-email
+++ b/contrib/hooks/post-receive-email
@@ -709,7 +709,7 @@ if [ -z "$GIT_DIR" ]; then
 	exit 1
 fi
 
-projectdesc=$(sed -ne '1p' "$GIT_DIR/description")
+projectdesc=$(sed -ne '1p' "$GIT_DIR/description" 2>/dev/null)
 # Check if the description is unchanged from it's default, and shorten it to
 # a more manageable length if it is
 if expr "$projectdesc" : "Unnamed repository.*$" >/dev/null
-- 
1.7.3.4

```