From: Sven Wischnowsky <wischnow@informatik.hu-berlin.de>
To: zsh-workers@sunsite.auc.dk
Subject: RE: GNU diff completion problem
Date: Fri, 10 Mar 2000 10:01:09 +0100 (MET) [thread overview]
Message-ID: <200003100901.KAA02760@beta.informatik.hu-berlin.de> (raw)
In-Reply-To: "Andrej Borsenkow"'s message of Fri, 10 Mar 2000 10:02:58 +0300
Andrej Borsenkow wrote:
> ...
>
> > } I have system default diff and GNU diff as gdiff. Current state does not
> > } allow me to complete GNU diff at all
> >
> > You want
> >
> > compdef _diff gdiff
> > zstyle ':completion:*:*:gdiff:*:version' command gdiff -v
> >
>
> Nope. I do want
>
> compdef _diff diff gdiff
>
> without bothering, if GNU diff is installed as main or additional program. The above will
> cover 99.9% of installations.
>
> I agree with the point, that _diff_options can be called in context without actual
> command. But it simply means, that _diff_options should not test diff version itself, but
> just get a "diff kind" parameter. Actual testing is to be done by _diff.
No it isn't. Or else we would have to do it in every function that
calls _diff_options.
How about this patch? It makes _diff_is_gnu an assoc of $words[1] and
it makes $words[1] be used as the command if it matches *diff. That
last part -- or at least the pattern -- may have to be changed: are
there other commands ending in `diff' that are not real diffs and that
may be in $words[1] when _diff_options is used?
Bye
Sven
diff -ru ../z.old/Completion/User/_diff_options Completion/User/_diff_options
--- ../z.old/Completion/User/_diff_options Thu Mar 9 15:47:15 2000
+++ Completion/User/_diff_options Fri Mar 10 09:53:31 2000
@@ -1,14 +1,24 @@
#autoload
-local of ofwuc ouc oss ofwy ofwg ofwl
+local of ofwuc ouc oss ofwy ofwg ofwl cmd
-(( $+_diff_is_gnu )) || {
- _diff_is_gnu=0;
- [[ $(_call version diff -v </dev/null 2>/dev/null) == *GNU* ]] && _diff_is_gnu=1
-}
+(( $+_diff_is_gnu )) || typeset -gA _diff_is_gnu
-if (( _diff_is_gnu ))
-then
+if (( ! $+_diff_is_gnu[$words[1]] )); then
+ if [[ "$words[1]" = *diff ]]; then
+ cmd="$words[1]"
+ else
+ cmd=diff
+ fi
+
+ if [[ $(_call version $cmd -v </dev/null 2>/dev/null) = *GNU* ]]; then
+ _diff_is_gnu[$words[1]]=yes
+ else
+ _diff_is_gnu[$words[1]]=
+ fi
+fi
+
+if [[ -n "$_diff_is_gnu[$words[1]]" ]]; then
# output formats
of="-y --side-by-side -n --rcs -e -f --ed -q --brief -c -C --context -u -U \
--unified --old-group-format --new-group-format --changed-group-format \
--
Sven Wischnowsky wischnow@informatik.hu-berlin.de
next reply other threads:[~2000-03-10 9:01 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2000-03-10 9:01 Sven Wischnowsky [this message]
2000-03-10 11:24 ` Bart Schaefer
2000-03-10 12:52 ` Alexandre Duret-Lutz
-- strict thread matches above, loose matches on Subject: below --
2000-03-09 16:45 Andrej Borsenkow
2000-03-09 17:30 ` Alexandre Duret-Lutz
2000-03-09 18:22 ` Bart Schaefer
2000-03-10 7:02 ` Andrej Borsenkow
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200003100901.KAA02760@beta.informatik.hu-berlin.de \
--to=wischnow@informatik.hu-berlin.de \
--cc=zsh-workers@sunsite.auc.dk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).