On 12/10/20 12:22 pm, Eli Schwartz wrote: > We leaked fullver and pkgarch all over the place, and only conditionally > unset the other variables. Marking them local is a more proactive > solution. >
OK. > Signed-off-by: Eli Schwartz <eschwa...@archlinux.org> > --- > scripts/makepkg.sh.in | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in > index a9e7c691..89da3fab 100644 > --- a/scripts/makepkg.sh.in > +++ b/scripts/makepkg.sh.in > @@ -869,6 +869,7 @@ install_package() { > } > > check_build_status() { > + local fullver pkgarch allpkgbuilt somepkgbuilt > if (( ! SPLITPKG )); then > fullver=$(get_full_version) > pkgarch=$(get_pkg_arch) > @@ -911,7 +912,6 @@ check_build_status() { > exit $E_ALREADY_BUILT > fi > fi > - unset allpkgbuilt somepkgbuilt > fi > } > >