zsh-workers
 help / color / mirror / code / Atom feed
* Debian _deb_packages_update_uninstalled() doesn't use cache?
@ 2008-07-07  8:36 Matt Wozniski
  2008-08-16 23:26 ` Matt Wozniski
  2008-08-17 13:34 ` Clint Adams
  0 siblings, 2 replies; 3+ messages in thread
From: Matt Wozniski @ 2008-07-07  8:36 UTC (permalink / raw)
  To: zsh-workers

As the subject indicates, this is a problem I have with a Debian
specific completer, but, since it's in CVS, I figured this is the
right place to ask... Let me know if I'm wrong.

Anyway, if I'm reading these functions right, then
_deb_packages_update_avail() and _deb_packages_update_installed() use
the cache, but _deb_packages_update_uninstalled() doesn't - even
though depends upon the cached data of _update_avail() and
_update_installed().  _update_uninstalled() seems to just be filtering
out the available packages (as found by _update_avail()) that are also
installed (as found by _update_installed()) so that they aren't
suggested as possible packages to install.  It seems to me that we're
not caching the operation that's most time consuming, even though it's
outcome is based entirely upon two operations whose outcomes are
cached.  As a result, completion for "aptitude install" (which uses
"_deb_packages uninstalled") is much slower than for "apt-get install"
(which uses "_deb_packages available") on my machine, at least for the
first completion in a newly opened shell.

Anyone have any insight to why _deb_packages_update_uninstalled()
might be doing this, or is this a bug?

~Matt


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Debian _deb_packages_update_uninstalled() doesn't use cache?
  2008-07-07  8:36 Debian _deb_packages_update_uninstalled() doesn't use cache? Matt Wozniski
@ 2008-08-16 23:26 ` Matt Wozniski
  2008-08-17 13:34 ` Clint Adams
  1 sibling, 0 replies; 3+ messages in thread
From: Matt Wozniski @ 2008-08-16 23:26 UTC (permalink / raw)
  To: zsh-workers

Ping?

On Mon, Jul 7, 2008 at 4:36 AM, Matt Wozniski wrote:
> As the subject indicates, this is a problem I have with a Debian
> specific completer, but, since it's in CVS, I figured this is the
> right place to ask... Let me know if I'm wrong.
>
> Anyway, if I'm reading these functions right, then
> _deb_packages_update_avail() and _deb_packages_update_installed() use
> the cache, but _deb_packages_update_uninstalled() doesn't - even
> though depends upon the cached data of _update_avail() and
> _update_installed().  _update_uninstalled() seems to just be filtering
> out the available packages (as found by _update_avail()) that are also
> installed (as found by _update_installed()) so that they aren't
> suggested as possible packages to install.  It seems to me that we're
> not caching the operation that's most time consuming, even though it's
> outcome is based entirely upon two operations whose outcomes are
> cached.  As a result, completion for "aptitude install" (which uses
> "_deb_packages uninstalled") is much slower than for "apt-get install"
> (which uses "_deb_packages available") on my machine, at least for the
> first completion in a newly opened shell.
>
> Anyone have any insight to why _deb_packages_update_uninstalled()
> might be doing this, or is this a bug?
>
> ~Matt
>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Debian _deb_packages_update_uninstalled() doesn't use cache?
  2008-07-07  8:36 Debian _deb_packages_update_uninstalled() doesn't use cache? Matt Wozniski
  2008-08-16 23:26 ` Matt Wozniski
@ 2008-08-17 13:34 ` Clint Adams
  1 sibling, 0 replies; 3+ messages in thread
From: Clint Adams @ 2008-08-17 13:34 UTC (permalink / raw)
  To: Matt Wozniski; +Cc: zsh-workers

On Mon, Jul 07, 2008 at 04:36:14AM -0400, Matt Wozniski wrote:
> Anyone have any insight to why _deb_packages_update_uninstalled()
> might be doing this, or is this a bug?

Seems reasonable to cache there as well.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-08-17 13:35 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-07-07  8:36 Debian _deb_packages_update_uninstalled() doesn't use cache? Matt Wozniski
2008-08-16 23:26 ` Matt Wozniski
2008-08-17 13:34 ` Clint Adams

Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).