On 27/02/11 21:48, Xavier Chantry wrote:
On Sun, Feb 27, 2011 at 12:40 PM, Allan McRae<allan@archlinux.org> wrote:
How much of a performance hit would there really be? If I have this right in my head (big assumption), the file-lists for all packages are read in in one go. So all that we would have to do is loop over them checking for the empty directory before we add it to the removal list. Should be a minimal impact.... maybe...
I suppose the best way is to make a patch and see!
AFAIK the file lists currently do not need to be loaded with -R, so the impact would be loading them more than the checking itself. Then again, we have that cost for all -S operation already, so why not make -R as bad ? :)
strace tells me that I was wrong and you are right. So that is big overhead...
Anyway we will keep having problems until we have a single file for the local db as well.
Yes, we will. I might attempt to a gathering of the contenders for improving this post 3.5. Allan