Setting/Querying cache-policy (was: Re: sudo autocompletion)

[Daniel Shahaf <d.s@xxxxxxxxxxxxxxxxxx>]

Oliver Kiddle wrote on Tue, 11 Feb 2020 11:12 +0100:
> Daniel Shahaf wrote:
> > dana wrote on Mon, 10 Feb 2020 11:13 -0600:  
> > > +  zstyle -e ":completion:${curcontext%:}:*" command-path \
> > > +    'reply=( $path ${path/%\/bin//sbin} )'  
> 
> I don't like it when completion functions set zstyles. I know that the
> cache functions do it but even with one bad precedent I think it is
> better to not proliferate the practice further.

+1

Speaking of the cache-policy style, why do the following places use
":completion:*:*:$service:*" in the «zstyle -s» call?  Shouldn't that
use ${curcontext}?

% ag cache-policy Completion/ | grep -v curcontext | vipe
Completion/Debian/Type/_deb_packages:101:  zstyle -s ":completion:*:*:$service:*" cache-policy update_policy
Completion/Debian/Type/_deb_packages:103:    zstyle ":completion:*:*:$service:*" cache-policy _debs_caching_policy
Completion/Debian/Command/_apt:629:  zstyle -s ":completion:*:*:$service:*" cache-policy update_policy
Completion/Debian/Command/_apt:631:    zstyle ":completion:*:*:$service:*" cache-policy _apt_caching_policy
Completion/Darwin/Type/_retrieve_mac_apps:64:  zstyle -s ":completion:*:*:$service:*" cache-policy cache_policy
Completion/Darwin/Type/_retrieve_mac_apps:66:    zstyle ":completion:*:*:$service:*" cache-policy _mac_apps_caching_policy
Completion/Redhat/Command/_rpm:433:zstyle -s ":completion:*:*:rpm:*" cache-policy update_policy
Completion/Redhat/Command/_rpm:435:  zstyle ":completion:*:*:rpm:*" cache-policy _rpms_caching_policy
Completion/Unix/Command/_subversion:26:  zstyle -s ":completion:*:*:$service:*" cache-policy update_policy
Completion/Unix/Command/_subversion:28:    zstyle ":completion:*:*:$service:*" cache-policy _svn_caching_policy
Completion/Unix/Command/_gradle:13:zstyle -s ":completion:*:*:$service:*" cache-policy cache_policy || \
Completion/Unix/Command/_gradle:14:    zstyle ":completion:*:*:$service:*" cache-policy _gradle_caching_policy
Completion/Unix/Command/_composer:802:    zstyle -s ":completion:*:*:$service:*" cache-policy tmp
Completion/Unix/Command/_composer:804:    zstyle ":completion:*:*:$service:*" cache-policy __composer_cache_policy
Completion/Linux/Command/_modutils:18:  zstyle -s ":completion:*:*:$service:*" cache-policy update_policy
Completion/Linux/Command/_modutils:20:    zstyle ":completion:*:*:$service:*" cache-policy _modules_caching_policy
Completion/Linux/Command/_opkg:301:  zstyle -s ":completion:*:*:$service:*" cache-policy cache_policy
Completion/Linux/Command/_opkg:303:  zstyle ":completion:*:*:$service:*" cache-policy __opkg_cache_policy

Cheers,

Daniel