zsh-workers
 help / color / mirror / code / Atom feed
* PATCH: 3.1.6-pws-2: _gv
@ 1999-09-01 14:46 Peter Stephenson
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Stephenson @ 1999-09-01 14:46 UTC (permalink / raw)
  To: Zsh hackers list

Some typos crept into _gv (at least the one I have, which is the one you
get).

Also, completion for configure failed when I used it via a path beginning
with a ~.  Apparently something (_arguments, presumably) tried to append
the current directory to the path, which didn't work awfully well.

(By the way, Sven, now I've thought of it: when you generate new completion
functions, could you use a directory name like Completion with a suffix
rather than oc for the old directory, since patch -p0 prefers the shorter
name and creates it in oc.  If you want to avoid having to type the whole
of Completion..., I believe there are now shells which can help in this
respect...)

*** Completion/User/_gv~	Mon Aug 30 16:18:56 1999
--- Completion/User/_gv	Wed Sep  1 16:35:24 1999
***************
*** 3,14 ****
  _xt_arguments \
    -{,no}{safer,quiet,center,swap,antialias,dsc,eof,pixmap,watch,resize} \
    -{monochrome,grayscale,color,portrait,landscape,upsidedown,seascape,h,help,v,spartan} \
- 
    '-arguments:ghostscript arguments:' \
    '-page:label of first page:' \
    '-media:page size:(Letter Legal Statement Tabloid Ledger Folio Quarto 10x14 Executive A3 A4 A5 B4 B5)' \
    '-scale:scale entry:' \
    '-scalebase:scale base:' \
    '-ad:resource file:_files' \
!   '-style:resource file:_files'
    '*:file:_pspdf'
--- 3,13 ----
  _xt_arguments \
    -{,no}{safer,quiet,center,swap,antialias,dsc,eof,pixmap,watch,resize} \
    -{monochrome,grayscale,color,portrait,landscape,upsidedown,seascape,h,help,v,spartan} \
    '-arguments:ghostscript arguments:' \
    '-page:label of first page:' \
    '-media:page size:(Letter Legal Statement Tabloid Ledger Folio Quarto 10x14 Executive A3 A4 A5 B4 B5)' \
    '-scale:scale entry:' \
    '-scalebase:scale base:' \
    '-ad:resource file:_files' \
!   '-style:resource file:_files' \
    '*:file:_pspdf'

-- 
Peter Stephenson <pws@ibmth.df.unipi.it>       Tel: +39 050 844536
WWW:  http://www.ifh.de/~pws/
Dipartimento di Fisica, Via Buonarroti 2, 56127 Pisa, Italy


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

* Re: PATCH: 3.1.6-pws-2: _gv
@ 1999-09-02  8:29 Sven Wischnowsky
  0 siblings, 0 replies; 2+ messages in thread
From: Sven Wischnowsky @ 1999-09-02  8:29 UTC (permalink / raw)
  To: zsh-workers


Peter Stephenson wrote:

> Some typos crept into _gv (at least the one I have, which is the one you
> get).

Oops.

> Also, completion for configure failed when I used it via a path beginning
> with a ~.  Apparently something (_arguments, presumably) tried to append
> the current directory to the path, which didn't work awfully well.

Yep, the caching could be improved, but the problem here was that
there were too many double quotes and a missing tilde (to have the
tilde expanded).

> (By the way, Sven, now I've thought of it: when you generate new completion
> functions, could you use a directory name like Completion with a suffix
> rather than oc for the old directory, since patch -p0 prefers the shorter
> name and creates it in oc.  If you want to avoid having to type the whole
> of Completion..., I believe there are now shells which can help in this
> respect...)

Cool. Where can I get them?

(And, yes, I can do that.)

> *** Completion/User/_gv~	Mon Aug 30 16:18:56 1999

Why is this in `User', btw? ghostview without X doesn't seem to make
much sense to me (ghostscript without X is ok, though).

Bye
 Sven

P.S.: I don't need the smiley, yes?

diff -u Completion.old/Base/_arguments Completion/Base/_arguments
--- Completion.old/Base/_arguments	Wed Sep  1 14:24:49 1999
+++ Completion/Base/_arguments	Thu Sep  2 10:27:12 1999
@@ -43,10 +43,9 @@
       tmpargv=( "${(@)argv[1,nth-1]}" )
     fi
 
-    if [[ "$words[1]" = /* ]]; then
-      tmp="$words[1]"
-    else
-      tmp="$PWD/$words[1]"
+    tmp=${~words[1]}
+    if [[ "$tmp" != /* ]]; then
+      tmp="$PWD/$tmp"
     fi
 
     if [[ "$tmp" != "$_args_cache_longcmd" ]]; then
@@ -92,7 +91,7 @@
       # those hyphens and anything from the space or comma after the
       # option up to the end. 
 
-      lopts=("--${(@)^${(@)${(@)${(@M)${(@ps:\n:j:\n:)${(@)${(@M)${(@f)$("$words[1]" --help 2>&1)//\[--/
+      lopts=("--${(@)^${(@)${(@)${(@M)${(@ps:\n:j:\n:)${(@)${(@M)${(@f)$(${~words[1]} --help 2>&1)//\[--/
 --}:#[ 	]#-*}//,/
 }}:#[ 	]#--*}#*--}%%[, ]*}:#}")
 

--
Sven Wischnowsky                         wischnow@informatik.hu-berlin.de


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

end of thread, other threads:[~1999-09-02  8:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1999-09-01 14:46 PATCH: 3.1.6-pws-2: _gv Peter Stephenson
1999-09-02  8:29 Sven Wischnowsky

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).