summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLuke Shumaker <lukeshu@parabola.nu>2017-09-17 00:00:11 +0200
committerJan Alexander Steffens (heftig) <jan.steffens@gmail.com>2017-10-30 15:59:38 +0100
commit095e5305e45a32d4eee1e43a493200f4bc8455b3 (patch)
treebdc2adee8350a22666ce5637d4062764cfcb74e7
parente4db687d108d7e7cd32654b9c8eed4c7c12cb094 (diff)
downloaddevtools-095e5305e45a32d4eee1e43a493200f4bc8455b3.tar.gz
devtools-095e5305e45a32d4eee1e43a493200f4bc8455b3.tar.xz
makechrootpkg: Fix function usage comments
A couple of the comments noting which globals are used by functions are outdated/wrong. - download_sources() : Remove USER from the list. It was always wrong. Originally, it should have been SUDO_USER (not USER), but I should have removed it entirely in 4f23609. - move_products() : Add SRCPKGDEST to the list. Though the commit adding the comment was only recently upstreamed (as 2fd5931), it originated in 2013 in a commit that has since been rebased many times. Anyway, in this rebasing, it missed move_products() starting to pay attention to SRCPKGDEST in fd1be1b (since nothing made git think there was a "conflict").
-rw-r--r--makechrootpkg.in2
1 files changed, 1 insertions, 1 deletions
diff --git a/makechrootpkg.in b/makechrootpkg.in
index f81c47e..d4c293f 100644
--- a/makechrootpkg.in
+++ b/makechrootpkg.in
@@ -245,7 +245,6 @@ _chrootnamcap() {
# Usage: download_sources $copydir $makepkg_user
# Globals:
# - SRCDEST
-# - USER
download_sources() {
local copydir=$1
local makepkg_user=$2
@@ -267,6 +266,7 @@ download_sources() {
# Globals:
# - PKGDEST
# - LOGDEST
+# - SRCPKGDEST
move_products() {
local copydir=$1
local src_owner=$2