Re: [PATCH] PR29022: 000-permissions files cause problems for backups

2022-04-13 Thread Aaron Merey via Elfutils-devel
Hi Mark, Thanks for the review. I fixed the issues you pointed out (good catch re. setting rc before closing fd) and pushed as commit 8b568fdea8 with Tested-by: Milian Wolff added. Milian, thanks again for taking a look at this. Aaron

Re: [PATCH] PR29022: 000-permissions files cause problems for backups

2022-04-13 Thread Mark Wielaard
Hi Aaron, On Fri, 2022-04-08 at 19:37 -0400, Aaron Merey via Elfutils-devel wrote: > I've revised this patch so that the negative-hit file's mtime is used > to calculate time since last download attempt instead of the cache_miss_s > file. I've also added a check for old 000-permission files so tha

Re: [PATCH] PR29022: 000-permissions files cause problems for backups

2022-04-09 Thread Milian Wolff
On Samstag, 9. April 2022 01:37:11 CEST Aaron Merey via Elfutils-devel wrote: > I've revised this patch so that the negative-hit file's mtime is used > to calculate time since last download attempt instead of the cache_miss_s > file. I've also added a check for old 000-permission files so that they

Re: [PATCH] PR29022: 000-permissions files cause problems for backups

2022-04-08 Thread Aaron Merey via Elfutils-devel
I've revised this patch so that the negative-hit file's mtime is used to calculate time since last download attempt instead of the cache_miss_s file. I've also added a check for old 000-permission files so that they are unlinked immediately if found. Aaron --- PR29022: 000-permissions files cause