[PATCH setup] Ignore reinstall requests if version is not accessible

[Christian Franke]

Jon Turney wrote:
> On 11/12/2022 15:11, Christian Franke via Cygwin-apps wrote:
>> This prevents accidental package loss if reinstall is run from local 
>> directory but the package is no longer cached.
>
> Thanks.  This seems right, so please apply.

Done.

Could you please also review "[PATCH setup] Add tooltip for Ctrl+I/R/U 
accelerator keys" before next release?


>
>> Possible further improvement: Do not offer "Reinstall" in the popup 
>> menu in this case.
>
> Indeed.
>
> That might be simply fixable, but really, we need to keep an index of 
> the files in the package cache, so we can handle situations like this 
> properly, without spending lots of time spinning rust 

Agree. "Install from Local Directory" is in particular slow on HDDs when 
directory tree is fragmented or even slower if the cache is on a network 
share.


> (we could then also do some expiry on the cache, which is a feature 
> that has been needed for decades...)


For example: Add checkbox(es) to LocalDirPage to enable/configure cache 
cleanup. In a later step remove all files that could no longer be 
downloaded (like Debian's "apt-get autoclean") and are no longer 
installed. The latter would need some heuristics for source packages as 
these do not appear in installed.db.

BTW, I have a local bash-script 'cygcache' which allows to list package 
states ([auto]installed, cached, curr/prev/test), cleanup the cache 
using above rules (configurable) or merge cache directories into one. 
Still requires some polishing. If there is any interest, I could ITP it 
next year.