[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Emacs-diffs] scratch/package-fix e5d5cdf 1/2: emacs-lisp/package.el
From: |
chad |
Subject: |
Re: [Emacs-diffs] scratch/package-fix e5d5cdf 1/2: emacs-lisp/package.el: Indicate incompatible packages. |
Date: |
Thu, 12 Feb 2015 13:42:09 -0800 |
> On 12 Feb 2015, at 06:41, Stefan Monnier <address@hidden> wrote:
>>>
>> Ideally, yes. That's why I factored it out as a function.
>> However, this is called on each package about to be printed, and checking
>> whether the dependencies are available imposes a performance cost that I'm
>> not sure we should pay. The time for printing the package list will go from
>> linear to quadratic in the number of packages.
>
> I understand, I was just pointing out that this current solution
> is incomplete. I expect that it's sufficient for now. […]
We could also make the recursive check happen only package.el tries
to do anything with a specific package (package-menu-mark-*,
package-menu-describe-package, etc). I don’t know if that’s better or
not.
~Chad