From b2c680bb02720f2a0c917d0e355c13d67bb89a4e Mon Sep 17 00:00:00 2001 From: Luke Shumaker Date: Fri, 22 Jun 2018 13:23:27 -0400 Subject: Minor cosmetic improvements - db-functions:set_repo_permission(): Align arguments to chgrp - db-move: Reword a comment - test: db-update: Fix typo in test name: "aregular" -> "a regular" --- db-functions | 2 +- db-move | 2 +- test/cases/db-update.bats | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/db-functions b/db-functions index 20a3267..d91a7f2 100644 --- a/db-functions +++ b/db-functions @@ -426,7 +426,7 @@ set_repo_permission() { if [[ -w ${dbfile} ]]; then local group=$(/usr/bin/stat --printf='%G' "$(dirname "${dbfile}")") - chgrp "$group" "${dbfile}" || error "Could not change group of %s to %s" "$dbfile" "$group" + chgrp "$group" "${dbfile}" || error "Could not change group of %s to %s" "$dbfile" "$group" chgrp "$group" "${filesfile}" || error "Could not change group of %s to %s" "$filesfile" "$group" chmod g+w "${dbfile}" || error "Could not set write permission for group %s to %s" "$group" "$dbfile" chmod g+w "${filesfile}" || error "Could not set write permission for group %s to %s" "$group" "$filesfile" diff --git a/db-move b/db-move index b02bc44..352af47 100755 --- a/db-move +++ b/db-move @@ -24,7 +24,7 @@ for pkgarch in "${ARCHES[@]}"; do repo_lock "${repo_from}" "${pkgarch}" || exit 1 done -# check if packages to be moved exist in vcs and ftp dir +# First loop is to check that all necessary files exist for pkgbase in "${args[@]:2}"; do found=false for tarch in "${ARCHES[@]}"; do diff --git a/test/cases/db-update.bats b/test/cases/db-update.bats index a4ca093..e805f99 100755 --- a/test/cases/db-update.bats +++ b/test/cases/db-update.bats @@ -224,7 +224,7 @@ load ../lib/common checkRemovedPackageDB extra 'pkg-any-b' } -@test "package has to be aregular file" { +@test "package has to be a regular file" { local p local target=$(mktemp -d) -- cgit v1.2.2