Re: [pacman-dev] [PATCH 1/2] contrib: Add verify-pacman-repo-db.pl

2016-08-06 Thread Allan McRae
On 19/07/16 00:28, Florian Pritz wrote: > Signed-off-by: Florian Pritz A commit message would be nice... Is there any reason PGP checksums are not checked? > --- > contrib/doc/verify-pacman-repo-db.1.txt | 50 ++ > contrib/verify-pacman-repo-db.pl| 279 > +

Re: [pacman-dev] [PATCH] Clarify alpm-hooks(5)

2016-08-06 Thread Allan McRae
On 24/07/16 06:15, Přemysl Janouch wrote: > While studying gtk-update-icon-cache.hook, which had the following contents: > > Target = usr/share/icons/*/ > Target = !usr/share/icons/*/?* > > it was not obvious from the documentation what and how this matches, > which I'm trying to rectify here

Re: [pacman-dev] [PATCH] can_remove_file: do not follow symlinks

2016-08-06 Thread Allan McRae
On 03/08/16 08:43, Joseph Burt wrote: > The symlink itself should be checked, not its target. > > Signed-off-by: Joseph Burt > --- > > This check also does more harm than good at the moment. If it fails once, > the rest of the package files are silently left on the filesystem. > Without this c

Re: [pacman-dev] [PATCH] Normalize alpm download callback's frontend cb arguments

2016-08-06 Thread Allan McRae
On 09/07/16 13:11, ivy.fos...@gmail.com wrote: > From: Ivy Foster > > When curl calls alpm's dlcb, alpm calls the frontend's cb with the > following (dlsize, totalsize) arguments: > > 0, -1: initialize > 0, 0: no change since last call > x {x>0, x0}: data downloaded, total size known > x {x>0},