[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu-web-doc-update: exit immediately when CVSROOT environmen
From: |
Pádraig Brady |
Subject: |
Re: [PATCH] gnu-web-doc-update: exit immediately when CVSROOT environment is missing |
Date: |
Tue, 27 Oct 2015 10:25:57 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 26/10/15 21:54, Sami Kerola wrote:
> * build-aux/gnu-web-doc-update: Check the environment variable CVSROOT early
> on to avoid needing to build a project before getting "cvs add: No CVSROOT
> specified! Please use the `-d' option" error.
> ---
> build-aux/gnu-web-doc-update | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/build-aux/gnu-web-doc-update b/build-aux/gnu-web-doc-update
> index c8fc7e5..b7bc254 100755
> --- a/build-aux/gnu-web-doc-update
> +++ b/build-aux/gnu-web-doc-update
> @@ -134,6 +134,9 @@ done
> test $# = 0 \
> || die "too many arguments"
>
> +test "x$CVSROOT" = "x" \
> + && die "environment variable \$CVSROOT not set"
> +
> prev=.prev-version
> version=$(cat $prev) || die "no $prev file?"
> pkg=$(sed -n 's/^PACKAGE = \(.*\)/\1/p' $builddir/Makefile) \
>
I don't remember having to set that.
Is there a more direct way like early `cvs co` or something?
thanks,
Pádraig