[Bash-completion-devel] [bash-completion-Bugs][314393] _known_hosts_real does not complete hostnames starting with a 't'

[bash-completion-bugs at alioth.debian.org]

bash-completion-Bugs item #314393 was changed at 2013-08-26 20:47 by Ville Skyttä
You can respond by visiting: 
https://alioth.debian.org/tracker/?func=detail&atid=413095&aid=314393&group_id=100114

Status: Open
Priority: 3
Submitted By: Rainer Müller (raimue-guest)
Assigned to: Nobody (None)
Summary: _known_hosts_real does not complete hostnames starting with a 't' 
Distribution: None
Originally reported in: None
Milestone: None
>Status: Fix Committed
Original bug number: 


Initial Comment:
There is a portability issue with a sed expression used in _known_hosts_real(). In BSD sed, the sequence '\t' does not represent a tab character, but it is recognized as a literal 't'. In Any hostname starting with a 't' was filtered on systems using BSD sed (in this case Mac OS X). GNU sed added '\t' as an extension and matches this as a tab character, which was intended here.

Attached is a patch that changes the occurrence in this sed expression to [[:space:]], which is less restrictive but should be okay in this context.

Original Report: http://trac.macports.org/ticket/32053

----------------------------------------------------------------------

>Comment By: Ville Skyttä  (scop-guest)
Date: 2013-08-26 20:47

Message:
The latter change is almost certainly incorrect, $'[:space:]' doesn't make sense to me. The point of $'\t' is to pre-expand the tab using bash, and that approach should work -- we use it a lot.

Could you verify if this works for you?
http://anonscm.debian.org/gitweb/?p=bash-completion/bash-completion.git;a=commitdiff;h=50ae57927365a16c830899cc1714be73237bdcb2

----------------------------------------------------------------------

You can respond by visiting: 
https://alioth.debian.org/tracker/?func=detail&atid=413095&aid=314393&group_id=100114