diff options
| author | Luke Shumaker <lukeshu@parabola.nu> | 2017-09-17 00:00:11 +0200 |
|---|---|---|
| committer | Jan Alexander Steffens (heftig) <jan.steffens@gmail.com> | 2017-10-30 15:59:38 +0100 |
| commit | 095e5305e45a32d4eee1e43a493200f4bc8455b3 (patch) | |
| tree | bdc2adee8350a22666ce5637d4062764cfcb74e7 | |
| parent | e4db687d108d7e7cd32654b9c8eed4c7c12cb094 (diff) | |
| download | devtools-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.in | 2 |
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 |