summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Armak <danarmak@gentoo.org>2003-07-04 13:21:02 +0000
committerDan Armak <danarmak@gentoo.org>2003-07-04 13:21:02 +0000
commit3be90aa84cb994c46ebba50e4067c7b794ce11ea (patch)
treecfa2cd319c6778289b0a2088b1601e82d0d726a3 /eclass/cvs.eclass
parentMoved stuff from body into src_compile, perhaps this helps with bug 23942 (diff)
downloadhistorical-3be90aa84cb994c46ebba50e4067c7b794ce11ea.tar.gz
historical-3be90aa84cb994c46ebba50e4067c7b794ce11ea.tar.bz2
historical-3be90aa84cb994c46ebba50e4067c7b794ce11ea.zip
fix ECVS_LOCALNAME handling for cases where ebuild calls cvs_src_unpack more than once. this happen in eg kdenonbeta modules automated via kde-source.eclass.
Diffstat (limited to 'eclass/cvs.eclass')
-rw-r--r--eclass/cvs.eclass22
1 files changed, 19 insertions, 3 deletions
diff --git a/eclass/cvs.eclass b/eclass/cvs.eclass
index f06cd0478ed8..2a5e02ffd278 100644
--- a/eclass/cvs.eclass
+++ b/eclass/cvs.eclass
@@ -1,6 +1,6 @@
# Copyright 1999-2003 Gentoo Technologies, Inc.
# Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/eclass/cvs.eclass,v 1.43 2003/06/30 18:17:11 danarmak Exp $
+# $Header: /var/cvsroot/gentoo-x86/eclass/cvs.eclass,v 1.44 2003/07/04 13:21:02 danarmak Exp $
#
# Author Dan Armak <danarmak@gentoo.org>
#
@@ -35,7 +35,7 @@ INHERITED="$INHERITED $ECLASS"
# like useless empty directories.
# WARNING: to be set only from within ebuilds! if set in your shell or some such,
# things wil break because the ebuild won't expect it and have e.g. a wrong $S setting.
-[ -z "$ECVS_LOCALNAME" ] && ECVS_LOCALNAME="$ECVS_MODULE"
+# ECVS_LOCALNAME
# Where the cvs modules are stored/accessed
[ -z "$ECVS_TOP_DIR" ] && ECVS_TOP_DIR="${DISTDIR}/cvs-src"
@@ -240,13 +240,24 @@ ECVS_TOP_DIR=$ECVS_TOP_DIR
ECVS_SERVER=$ECVS_SERVER
ECVS_USER=$ECVS_USER
ECVS_PASS=$ECVS_PASS
-ECS_MODULE=$ECVS_MODULE
+ECVS_MODULE=$ECVS_MODULE
ECVS_LOCAL=$ECVS_LOCAL
ECVS_RUNAS=$ECVS_RUNAS
ECVS_LOCALNAME=$ECVS_LOCALNAME"
[ -z "$ECVS_MODULE" ] && die "ERROR: CVS module not set, cannot continue."
+ # merely setting this default value makes things fail when cvs_src_unpack is called
+ # more than once per ebuild (eg kdenonbeta submodules); so if we set a default value,
+ # we disable it again at the function's end.
+ # of course, we could instead always reference it with the bash syntax for 'take default
+ # value from this other variable if undefined', but i'm lazy.
+ if [ -z "$ECVS_LOCALNAME" ]; then
+ ECVS_LOCALNAME="$ECVS_MODULE"
+ ECVS_LOCALNAME_SETDEFAULT=true
+ fi
+
+
if [ "$ECVS_SERVER" == "offline" ]; then
# we're not required to fetch anything, the module already exists and shouldn't be updated
if [ -d "${ECVS_TOP_DIR}/${ECVS_LOCALNAME}" ]; then
@@ -295,6 +306,11 @@ ECVS_LOCALNAME=$ECVS_LOCALNAME"
# cvs_src_unpack is usually called several times from e.g. kde-source_src_unpack
export PATCHES=""
fi
+
+ if [ -n "$ECVS_LOCALNAME_SETDEFAULT" ]; then
+ unset ECVS_LOCALNAME
+ unset ECVS_LOCALNAME_SETDEFAULT
+ fi
}