1
0
mirror of https://github.com/moparisthebest/pacman synced 2024-08-13 17:03:46 -04:00

lint_pkgbuild: explicitly return value

Set the return value to be local and always explictly returns it.

Signed-off-by: Allan McRae <allan@archlinux.org>
This commit is contained in:
Allan McRae 2015-11-13 14:32:52 +10:00
parent 947dfda515
commit c26e5b81e5
5 changed files with 12 additions and 4 deletions

View File

@ -31,7 +31,7 @@ lint_pkgbuild_functions+=('lint_arch')
lint_arch() {
local a name list
local a name list ret=0
if [[ $arch == 'any' ]]; then
return 0

View File

@ -41,5 +41,7 @@ lint_changelog() {
fi
done
check_files_exist 'changelog' "${changelog_list[@]}"
check_files_exist 'changelog' "${changelog_list[@]}" || ret=1
return $ret
}

View File

@ -40,5 +40,7 @@ lint_install() {
install_list+=("$file")
done
check_files_exist 'install' "${install_list[@]}"
check_files_exist 'install' "${install_list[@]}" || ret=1
return $ret
}

View File

@ -30,6 +30,8 @@ lint_pkgbuild_functions+=('lint_pkgbase')
lint_pkgbase() {
local ret=0
if [[ ${pkgbase:0:1} = "-" ]]; then
error "$(gettext "%s is not allowed to start with a hyphen.")" "pkgname"
return 1
@ -43,4 +45,6 @@ lint_pkgbase() {
'pkgbase' "${i//[[:alnum:]+_.@-]}"
ret=1
fi
return $ret
}

View File

@ -27,7 +27,7 @@ source "$LIBRARY/util/message.sh"
check_files_exist() {
local kind=$1 files=("${@:2}") file ret
local kind=$1 files=("${@:2}") file ret=0
for file in "${files[@]}"; do
if [[ $file && ! -f $file ]]; then