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

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

2022-04-05 Thread Aaron Merey via Elfutils-devel
000-permission files currently used for negative caching can cause permission problems for some backup software and disk usage checkers. Fix this by using empty files to for negative caching instead. https://sourceware.org/bugzilla/show_bug.cgi?id=29022 Signed-off-by: Aaron Merey --- debuginfo