On 14/10/10 10:54, Allan McRae wrote:
On 14/10/10 08:53, Dan McGee wrote:
On Wed, Oct 13, 2010 at 5:36 PM, Xavier Chantry <chantry.xavier@gmail.com> wrote:
On Tue, Oct 12, 2010 at 12:38 AM, Dan McGee<dpmcgee@gmail.com> wrote:
Xavier and I talked about it, and my numbers were a bit flawed due to having a debug-compiled pacman and comparing it against the system pacman. We also figured it would help a lot on patches with a slow stat (Cygwin, etc.).
-Dan
Allan already killed it with hatred in the backend branch :) Maybe we want to keep this in be_local.c : _alpm_local_db_populate ? I still don't believe that avoiding thousands of stats can hurt, but probably no big deal.
Yes, we will probably want to port it into that set of patches, I agree.
Whoops! Poor rebasing on my part. I was intending to keep this iin be_local and has assumed it was... I will fix that.
And it has been restored. Although the majority of it use has been removed due to not reading in the sync database via a directory list on the file system. Allan