[Bash-completion-devel] Bug#487449: Fw: Bug#487449: bash-completion: Wrong directory name completion

[David Paleino]

Forwarding my reply to Nelson -- I had mistaken the BTS address....

Inizio messsaggio inviato:

Data: Sun, 22 Jun 2008 01:07:44 +0200
Da: David Paleino <d.paleino at gmail.com>
A: "Nelson A. de Oliveira" <naoliv at debian.org>, 487449.quiet at bugs.debian.org
Oggetto: Re: Bug#487449: bash-completion: Wrong directory name completion


On Sat, 21 Jun 2008 19:45:08 -0300, Nelson A. de Oliveira wrote:

> Package: bash-completion
> Version: 20080617.1
> Severity: normal
> 
> Hi again!

Since I won't sleep if you (or anyone else) are unable to use bash-completion
because of my lazy testing, here's a quick patch I believe fixes the problem
(r1121):

=== modified file 'bash_completion'
--- bash_completion	2008-06-21 22:51:09 +0000
+++ bash_completion	2008-06-21 23:05:48 +0000
@@ -241,13 +241,13 @@
 	#set -f		 # disable pathname expansion (globbing)
 
 	if [ "${1:-}" = -d ]; then
-		COMPREPLY=( "${COMPREPLY[@]:-}" $( compgen -d -- $cur ) )
+		COMPREPLY=( ${COMPREPLY[@]:-} $( compgen -d -- $cur ) )
 		#eval "$glob"    # restore glob setting.
 		return 0
 	fi
 
 	xspec=${1:+"!*.$1"}	# set only if glob passed in as $1
-	COMPREPLY=( "${COMPREPLY[@]:-}" $( compgen -f -X "$xspec" -- "$cur" ) \
+	COMPREPLY=( ${COMPREPLY[@]:-} $( compgen -f -X "$xspec" -- "$cur" ) \
 		    $( compgen -d -- "$cur" ) )
 	#eval "$glob"    # restore glob setting.
 }

Goodnight,
David (who is going to sleep... seriously.)

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 ----|---- http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/bash-completion-devel/attachments/20080622/c7a30f5b/attachment.pgp