March 2018 - Arch-projects - lists.archlinux.org

[arch-projects] [netctl] [GIT] Network control utility based on systemd branch master updated. 1.15-3-gcef231d
by Jouke Witteveen 16 Mar '18

16 Mar '18

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Network control utility based on systemd". The branch, master has been updated via cef231d762e7bb65bcf9a6a040ba367c5062a8f8 (commit) via c3b4f7515dfb78220528eb6c93931eec34aa9821 (commit) from 2d7ec0b8459c4d42b43da1b6727a91e701bfaadc (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit cef231d762e7bb65bcf9a6a040ba367c5062a8f8 Author: Erich Eckner <arch(a)eckner.net> Date: Fri Mar 16 12:02:45 2018 +0100 Fix quoting of After= in generated service files (FS#57850) commit c3b4f7515dfb78220528eb6c93931eec34aa9821 Author: Jouke Witteveen <j.witteveen(a)gmail.com> Date: Mon Feb 26 12:59:30 2018 +0100 Only wait when necessary in PPP cleanup code ----------------------------------------------------------------------- Summary of changes: src/lib/connections/mobile_ppp | 4 ++-- src/lib/connections/pppoe | 4 ++-- src/netctl.in | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) hooks/post-receive -- Network control utility based on systemd

1 0

[arch-projects] [dbscripts] [GIT] Official repo DB scripts branch master updated. 20131102-73-gda49ea6
by eschwartz＠archlinux.org 16 Mar '18

16 Mar '18

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Official repo DB scripts". The branch, master has been updated via da49ea616e162d3cfcb4b6a4242f497e94c36564 (commit) via 4ae3ea2f71344045fa4990c0524b4c662ab83cfc (commit) via 33aae318542016f66d2f0e09654606649d404ff2 (commit) via f22f1554720da245955337782ce554d5c3f93c8c (commit) from e53cad6e4a8284165c6d0b2c7c86f6c077be693b (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit da49ea616e162d3cfcb4b6a4242f497e94c36564 Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Thu Feb 22 21:15:40 2018 -0500 test: Fixup glob matching - ftpdir-cleanup.bats: Glob expansion does not occur in [[ -f ]] tests. The [[ ! -f .../${pkgname}-*${PKGEXT} ]] checks were checking that there were no files containing a literal '*' for that part of their name. Obviously, this isn't what was intended. - sourceballs.bats: [ -r ] checks explode if the glob returns >1 file. Avoid using them if the path being checked contains a glob. commit 4ae3ea2f71344045fa4990c0524b4c662ab83cfc Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Thu Feb 22 21:15:39 2018 -0500 test: common.bash:__getCheckSum: Don't rely on IFS I managed to stumble across a bug in BATS where the run() function screwed with the global IFS. The bug has been fixed in git, but isn't in a release yet. https://github.com/sstephenson/bats/issues/89 Anyway, this bug breaks __getCheckSum(). Fortunately, we have avoided tripping it so far because luck has it that we never call __getCheckSum() after run() in the same test. So, there's nothing actually broken here, but it makes me nervous. So go ahead and modify __getCheckSum to not rely on IFS. And, while we're at it: declare the result variable and set it as separate commands. Doing both in the same command masks the exit code of the subshell expansion. We don't explicitly check the exit code, but BATS runs the test suite with `set -e`, so splitting it does mean that BATS will now detect errors from sha1sum. We don't really expect that to happen, but if BATS will give us error checking on it for free, why not? commit 33aae318542016f66d2f0e09654606649d404ff2 Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Thu Feb 22 21:15:41 2018 -0500 Update messages to make fuller use of printf formatters These are things that were (IMO) missed in 5afac1e. I found them using: git grep -E '(plain|msg|msg2|warning|error|die) "[^"]*\$' I went a little above-and-beyond for escaping strings for the error messages in db-functions' arch_repo_add and arch_repo_remove. The code should explain itself, but I wanted to point it out, as it's more than the usual "slap %s in there, and move the ${...} to the right". commit f22f1554720da245955337782ce554d5c3f93c8c Author: Eli Schwartz <eschwartz(a)archlinux.org> Date: Mon Feb 26 17:57:31 2018 -0500 db-functions: unify the repo-add/repo-remove caller Use less duplicated logic, opening the way to more elegantly change the interface for the repo-add/repo-remove scripts which are themselves symlinked together and mostly have the same interface. ----------------------------------------------------------------------- Summary of changes: cron-jobs/ftpdir-cleanup | 8 ++++---- cron-jobs/integrity-check | 2 +- cron-jobs/sourceballs | 8 ++++---- db-functions | 34 ++++++++++++---------------------- db-move | 8 ++++---- db-remove | 4 ++-- db-repo-add | 4 ++-- db-repo-remove | 4 ++-- db-update | 4 ++-- test/cases/ftpdir-cleanup.bats | 4 ++-- test/cases/sourceballs.bats | 4 ++-- test/lib/common.bash | 15 +++++++++++++-- testing2x | 2 +- 13 files changed, 51 insertions(+), 50 deletions(-) hooks/post-receive -- Official repo DB scripts

1 0

[arch-projects] [dbscripts] [PATCH 0/8] Backports from Parabola
by Luke Shumaker 14 Mar '18

14 Mar '18

From: Luke Shumaker <lukeshu(a)parabola.nu> These are commits from Parabola's dbscripts that I think are upstreamable. No real theme to them, but because some of them are wide-sweeping (eg. clean up quoting), they are a good base layer to apply before backporting other changes. I've done my best to look over them, but at least one weird thing did happen while rebasing them, so it's possible I missed something that rebased weird. Some of this duplicates (non-backported) changes in the last patchset I submitted. BTW, now that dbscripts is on GitHub, is that the preferred way of submitting these? Or is this mailing list still best? Luke Shumaker (8): Fix quoting around variables, especially arrays. Use printf-formatters instead of string interpolation on msg, error, etc. Export TMPDIR, and use mktemp -t instead of making it part of the template Use `grep &>/dev/null` instead of `grep -q` when operating on piped stdin. Add "#!/hint/bash" to the beginning of several files. Consistently use "$(dirname "$(readlink -e "$0")")" Normalize to tab indent. devlist-mailer: Make LIST and FROM configurable config | 11 ++- config.local.svn-community | 2 + config.local.svn-packages | 2 + cron-jobs/check_archlinux/parse_pkgbuilds.sh | 64 ++++++++--------- cron-jobs/devlist-mailer | 13 ++-- cron-jobs/ftpdir-cleanup | 42 +++++------ cron-jobs/integrity-check | 8 +-- cron-jobs/sourceballs | 50 +++++++------- cron-jobs/update-web-db | 20 +++--- db-functions | 100 +++++++++++++-------------- db-move | 60 ++++++++-------- db-remove | 24 +++---- db-repo-add | 24 +++---- db-repo-remove | 24 +++---- db-update | 38 +++++----- test/cases/db-update.bats | 2 +- test/lib/common.bash | 6 +- testing2x | 36 +++++----- 18 files changed, 267 insertions(+), 259 deletions(-) -- Happy hacking, ~ Luke Shumaker

2 18

[arch-projects] [dbscripts] [PATCH] db-functions: unify the repo-add/repo-remove caller
by Eli Schwartz 14 Mar '18

14 Mar '18

Use less duplicated logic, opening the way to more elegantly change the interface for the repo-add/repo-remove scripts which are themselves symlinked together and mostly have the same interface. Signed-off-by: Eli Schwartz <eschwartz(a)archlinux.org> --- db-functions | 34 ++++++++++++---------------------- db-move | 4 ++-- db-remove | 2 +- db-repo-add | 2 +- db-repo-remove | 2 +- db-update | 2 +- 6 files changed, 18 insertions(+), 28 deletions(-) diff --git a/db-functions b/db-functions index 8b71cae..653f130 100644 --- a/db-functions +++ b/db-functions @@ -442,34 +442,24 @@ set_repo_permission() { fi } -arch_repo_add() { - local repo=$1 - local arch=$2 - local pkgs=(${@:3}) - - # package files might be relative to repo dir - pushd "${FTP_BASE}/${repo}/os/${arch}" >/dev/null - /usr/bin/repo-add -q "${repo}${DBEXT}" ${pkgs[@]} \ - || error "repo-add ${repo}${DBEXT} ${pkgs[@]}" - popd >/dev/null - set_repo_permission "${repo}" "${arch}" - - REPO_MODIFIED=1 -} - -arch_repo_remove() { - local repo=$1 - local arch=$2 - local pkgs=(${@:3}) +arch_repo_modify() { + local action=$1 + local repo=$2 + local arch=$3 + local pkgs=("${@:4}") local dbfile="${FTP_BASE}/${repo}/os/${arch}/${repo}${DBEXT}" - if [[ ! -f ${dbfile} ]]; then + if [[ ${action} = remove && ! -f ${dbfile} ]]; then error "No database found at '%s'" "$dbfile" return 1 fi - /usr/bin/repo-remove -q "${dbfile}" ${pkgs[@]} \ - || error "repo-remove ${dbfile} ${pkgs[@]}" + + # package files for repo-add might be relative to repo dir + pushd "${dbfile%/*}" >/dev/null + /usr/bin/repo-${action} -q "${dbfile}" ${pkgs[@]} \ + || error "repo-${action} ${dbfile} ${pkgs[@]}" set_repo_permission "${repo}" "${arch}" + popd >/dev/null REPO_MODIFIED=1 } diff --git a/db-move b/db-move index fb7ebac..e9c9fa7 100755 --- a/db-move +++ b/db-move @@ -114,8 +114,8 @@ done for tarch in ${ARCHES[@]}; do if [[ -n ${add_pkgs[${tarch}]} ]]; then - arch_repo_add "${repo_to}" "${tarch}" ${add_pkgs[${tarch}]} - arch_repo_remove "${repo_from}" "${tarch}" ${remove_pkgs[${tarch}]} + arch_repo_modify add "${repo_to}" "${tarch}" ${add_pkgs[${tarch}]} + arch_repo_modify remove "${repo_from}" "${tarch}" ${remove_pkgs[${tarch}]} fi done diff --git a/db-remove b/db-remove index 70502bc..08a95f9 100755 --- a/db-remove +++ b/db-remove @@ -47,6 +47,6 @@ for pkgbase in ${pkgbases[@]}; do done for tarch in ${tarches[@]}; do - arch_repo_remove "${repo}" "${tarch}" ${remove_pkgs[@]} + arch_repo_modify remove "${repo}" "${tarch}" ${remove_pkgs[@]} repo_unlock $repo $tarch done diff --git a/db-repo-add b/db-repo-add index d7be302..4b16f7e 100755 --- a/db-repo-add +++ b/db-repo-add @@ -36,6 +36,6 @@ for tarch in ${tarches[@]}; do msg "Adding %s to [%s]..." "$pkgfile" "$repo" fi done - arch_repo_add "${repo}" "${tarch}" ${pkgfiles[@]} + arch_repo_modify add "${repo}" "${tarch}" ${pkgfiles[@]} repo_unlock $repo $tarch done diff --git a/db-repo-remove b/db-repo-remove index 32d167e..3077786 100755 --- a/db-repo-remove +++ b/db-repo-remove @@ -32,6 +32,6 @@ for tarch in ${tarches[@]}; do for pkgname in ${pkgnames[@]}; do msg "Removing %s from [%s]..." "$pkgname" "$repo" done - arch_repo_remove "${repo}" "${tarch}" ${pkgnames[@]} + arch_repo_modify remove "${repo}" "${tarch}" ${pkgnames[@]} repo_unlock $repo $tarch done diff --git a/db-update b/db-update index 4e17184..e8ae08d 100755 --- a/db-update +++ b/db-update @@ -94,7 +94,7 @@ for repo in ${repos[@]}; do add_pkgs+=("${pkgfile}") done if (( ${#add_pkgs[@]} >= 1 )); then - arch_repo_add "${repo}" "${pkgarch}" ${add_pkgs[@]} + arch_repo_modify add "${repo}" "${pkgarch}" ${add_pkgs[@]} fi done done -- 2.16.2

2 1