diff options
author | Dan McGee <dan@archlinux.org> | 2007-06-02 13:39:47 -0400 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2007-06-02 13:44:54 -0400 |
commit | 214f264bd06afc0d1cc479c75e46b05cc0e85200 (patch) | |
tree | a23c0f87676d2c7947b52ed8a4c588aa460ef5bd /scripts | |
parent | 111c0f71c3d4154d2b6694c855f11f0d4b07cb9d (diff) |
Rename functions in makepkg for consistency
Add a hyphen to any function name that needs it so we are consistent across
the board.
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'scripts')
-rw-r--r-- | scripts/makepkg.in | 34 |
1 files changed, 17 insertions, 17 deletions
diff --git a/scripts/makepkg.in b/scripts/makepkg.in index 6ae4d8be..60b5fa28 100644 --- a/scripts/makepkg.in +++ b/scripts/makepkg.in @@ -177,7 +177,7 @@ in_array() { return 1 } -getdownloadclient() { +get_downloadclient() { # $1 = url with valid protocol prefix local url=$1 local proto=$(echo $netfile | sed 's|://.*||') @@ -211,7 +211,7 @@ getdownloadclient() { echo "$agent" } -checkdeps() { +check_deps() { [ $# -gt 0 ] || return pmout=$(pacman $PACMAN_OPTS -T $*) @@ -302,16 +302,16 @@ handledeps() { return $R_DEPS_SATISFIED } -resolvedeps() { +resolve_deps() { local R_DEPS_SATISFIED=0 local R_DEPS_MISSING=0 - deplist="$(checkdeps $*)" + deplist="$(check_deps $*)" [ "$deplist" = "" ] && return $R_DEPS_SATISFIED if handledeps $deplist; then # check deps again to make sure they were resolved - deplist="$(checkdeps $*)" + deplist="$(check_deps $*)" [ "$deplist" = "" ] && return $R_DEPS_SATISFIED elif [ "$DEP_BIN" = "1" -o "$DEP_SRC" = "1" ]; then error "$(gettext "Failed to install all missing dependencies.")" @@ -327,7 +327,7 @@ resolvedeps() { } # fix flyspray bug #5923 -removedeps() { +remove_deps() { [ "$RMDEPS" = "0" ] && return # runtimedeps and buildtimedeps are set when resolving deps @@ -365,7 +365,7 @@ download_sources() { fi # find the client we should use for this URL - local dlclient=$(getdownloadclient $netfile) || exit $? + local dlclient=$(get_downloadclient $netfile) || exit $? msg2 "$(gettext "Downloading %s")" "$file" # fix flyspray bug #3289 @@ -590,7 +590,7 @@ run_build() { if [ $ret -gt 0 ]; then error "$(gettext "Build Failed. Aborting...")" - removedeps + remove_deps exit 2 # $E_BUILD_FAILED # TODO: error code fi } @@ -828,7 +828,7 @@ create_srcpackage() { fi } -installpackage() { +install_package() { [ "$INSTALL" = "0" ] && return msg "$(gettext "Installing package with pacman -U...")" @@ -1128,7 +1128,7 @@ if [ -f "$PKGDEST/${pkgname}-${pkgver}-${pkgrel}-${CARCH}${PKGEXT}" \ -a "$FORCE" = "0" -a "$GENINTEG" = "0" -a "$SOURCEONLY" = "0" ]; then if [ "$INSTALL" = "1" ]; then warning "$(gettext "a package has already been built, installing existing package.")" - installpackage + install_package exit $? else error "$(gettext "a package has already been built. (use -f to overwrite)")" @@ -1178,22 +1178,22 @@ if [ "$NODEPS" = "1" -o "$GENINTEG" = "1" -o "$NOBUILD" = "1" -o "$REPKG" = "1" # skip printing a warning message for the others: geninteg, nobuild, repkg elif [ $(type -p pacman) ]; then deperr=0 - # these two variables are needed later by removedeps + # these two variables are needed later by remove_deps unset runtimedeps buildtimedeps msg "$(gettext "Checking Runtime Dependencies...")" - resolvedeps ${depends[@]} + resolve_deps ${depends[@]} ret=$? - # deplist is a global variable set by resolvedeps + # deplist is a global variable set by resolve_deps runtimedeps="$deplist" if [ "$ret" != "0" ]; then deperr=1 fi msg "$(gettext "Checking Buildtime Dependencies...")" - resolvedeps ${makedepends[@]} + resolve_deps ${makedepends[@]} ret=$? - # deplist is a global variable set by resolvedeps + # deplist is a global variable set by resolve_deps buildtimedeps="$deplist" if [ "$ret" != "0" ]; then deperr=1 @@ -1267,11 +1267,11 @@ if [ "$CLEANUP" = "1" ]; then rm -rf ${pkgname}-${pkgver}-${pkgrel}-${CARCH}.log* fi -removedeps +remove_deps msg "$(gettext "Finished making: %s")" "$pkgname ($(date))" -installpackage +install_package exit 0 #E_OK |