[PATCH] ant completion

[PATCH] ant completion

[Jean-Baptiste Quenot]

[-- Attachment #1: Type: text/plain, Size: 425 bytes --]

Dear workers,

Please find  attached a  patch for  ant completion.   The current
ant  completion  script  only  works when  the  "name"  attribute
immediately follows the "target" element name.

In  the following  example, zsh  only offers  completion for  the
'othertarget' target:

<target name='othertarget'/>
<target depends='othertarget' name='mytarget'/>

Best regards,
-- 
Jean-Baptiste Quenot
http://caraldi.com/jbq/

[-- Attachment #2: zsh-ant-comp-patch --]
[-- Type: text/plain, Size: 492 bytes --]

--- _ant.orig	Tue Aug 24 14:40:26 2004
+++ _ant	Tue Aug 24 14:51:38 2004
@@ -86,7 +86,7 @@
 	buildfile=${(v)opt_args[(I)(-f|-file|-buildfile)]:-build.xml}
       fi
       if [[ -f $buildfile ]]; then
-	targets=( $(sed -n 's/ *<target name=["'"'"']\([^"'"'"']*\)["'"'"'].*/\1/p' < $buildfile) )
+	targets=( $(sed -n 's/ *<target[^>]*name=["'"'"']\([^"'"'"']*\)["'"'"'].*/\1/p' < $buildfile) )
 	_wanted targets expl target compadd -a targets && ret=0
       else
 	_message -e targets target

Re: [PATCH] ant completion

[Wayne Davison]

On Tue, Aug 24, 2004 at 02:52:27PM +0200, Jean-Baptiste Quenot wrote:
> Please find  attached a  patch for  ant completion.   The current
> ant  completion  script  only  works when  the  "name"  attribute
> immediately follows the "target" element name.

Thanks -- I checked in your suggested change, though I did leave in the
space in front of the "name=" attribute, just for a little more paraoid
safety.  I.e.:

    ...<target[^>]* name=...

..wayne..