August 2017 - Pacman-contrib - lists.archlinux.org

[pacman-contrib] [GIT] branch master updated (8fd72f6 -> b61b9b2)
by demize＠archlinux.org 21 Aug '17

21 Aug '17

This is an automated email from the git hooks/post-receive script. demize pushed a change to branch master in repository pacman-contrib. from 8fd72f6 Remove --optdeps from pactree manpage new b61b9b2 checkupdates: report failure to fetch updates The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Graph of new commits: * b61b9b2 (HEAD -> master) checkupdates: report failure to fetch updates Detailed log of new commits: commit b61b9b25ac3d1d429e3967b95bd907cfc75836ef Author: Andrew Barchuk <andrew(a)raindev.io> Date: Mon Aug 21 22:05:38 2017 +0200 checkupdates: report failure to fetch updates Exit the script with 1 if pacman -Sy call failed. This would let a user to distinguish whether there're no updates or pacman failed to fetch them. Fixes FS#55206 Signed-off-by: Andrew Barchuk <andrew(a)raindev.io> Message-Id: <1503345938.476708.1080452304.09FD1100(a)webmail.messagingengine.com> Summary of changes: src/checkupdates.sh.in | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) -- To stop receiving notification emails like this one, please contact the administrator of this repository.

1 0

[pacman-contrib] [PATCH] checkupdates: report failure to fetch updates
by Andrew Barchuk 21 Aug '17

21 Aug '17

Exit the script with 1 if pacman -Sy call failed. This would let a user to distinguish whether there're no updates or pacman failed to fetch them. Fixes FS#55206 Signed-off-by: Andrew Barchuk <andrew(a)raindev.io> --- src/checkupdates.sh.in | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/checkupdates.sh.in b/src/checkupdates.sh.in index 4162548..8b9a292 100644 --- a/src/checkupdates.sh.in +++ b/src/checkupdates.sh.in @@ -53,7 +53,10 @@ fi mkdir -p "$CHECKUPDATES_DB" ln -s "${DBPath}/local" "$CHECKUPDATES_DB" &> /dev/null -fakeroot -- pacman -Sy --dbpath "$CHECKUPDATES_DB" --logfile /dev/null &> /dev/null +if ! fakeroot -- pacman -Sy --dbpath "$CHECKUPDATES_DB" --logfile /dev/null &> /dev/null; then + error 'Cannot fetch updates' + exit 1 +fi pacman -Qu --dbpath "$CHECKUPDATES_DB" 2> /dev/null | grep -v '\[.*\]' exit 0 -- 2.14.1

1 0

[pacman-contrib] checkupdates sets wrong exit status when network is down
by Andrew Barchuk 20 Aug '17

20 Aug '17

Greetings, I've discovered that checkupdates script ignores failures to fetch package updates silently and exits with 0. Basically result of pacman -Sy is not handled. Obviously it would be good to let the user know whether there're no package updates or it's not possible to check. I've opened an issue on Arch Linux bug tracker: https://bugs.archlinux.org/task/55206 Here's a possible diff to fix the problem: $ diff pacman-5.0.2/contrib/checkupdates.sh.in ~/checkupdates.sh.in 54c54,57 < fakeroot -- pacman -Sy --dbpath "$CHECKUPDATES_DB" --logfile /dev/null &> /dev/null --- > if ! fakeroot -- pacman -Sy --dbpath "$CHECKUPDATES_DB" --logfile /dev/null &> /dev/null; then > error 'Cannot fetch updates' > exit 1 > fi Thanks! --- Andrew

1 0