# Patch for git-completion.bash [SEC=UNCLASSIFIED]

3 messages from 2010-08-16 to 2010-08-16. Participants: David Scurrah, Ævar Arnfjörð Bjarmason.
Thread: https://gitlist.dev/t/24751

## David Scurrah, 2010-08-16 00:53

Subject: Patch for git-completion.bash [SEC=UNCLASSIFIED]
Message-ID: <0E3686EB9FA8AA409AFA0A25468DCE4301309C3DE865@BOM-VMBX-HO.bom.gov.au>
URL: https://gitlist.dev/e/0E3686EB9FA8AA409AFA0A25468DCE4301309C3DE865%40BOM-VMBX-HO.bom.gov.au

```



From 593b92c22fb72884c4d61e6a4ae1dd65db89425f Mon Sep 17 00:00:00 2001
From: David Scurrah <dscurrah@bom.gov.au>
Date: Mon, 16 Aug 2010 10:35:28 +1000
Subject: [PATCH] Completion script fails because of syntax error

---
 contrib/completion/git-completion.bash |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index 6756990..64b4f09 100755
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -120,7 +120,7 @@ __git_ps1_show_upstream ()
 			upstream=svn+git # default upstream is SVN if available, else git
 			;;
 		esac
-	done < <(git config -z --get-regexp '^(svn-remote\..*\.url|bash\.showupstream)$' 2>/dev/null | tr '\0\n' '\n ')
+	done < $(git config -z --get-regexp '^(svn-remote\..*\.url|bash\.showupstream)$' 2>/dev/null | tr '\0\n' '\n ')
 
 	# parse configuration values
 	for option in ${GIT_PS1_SHOWUPSTREAM}; do
-- 
1.7.0.1


```

## Ævar Arnfjörð Bjarmason, 2010-08-16 01:09

Subject: Re: Patch for git-completion.bash [SEC=UNCLASSIFIED]
Message-ID: <AANLkTimGrrvz6myV=Fmioqh9=rt+Fh7qSSLcoma6Z-Bw@mail.gmail.com>
URL: https://gitlist.dev/e/AANLkTimGrrvz6myV%3DFmioqh9%3Drt%2BFh7qSSLcoma6Z-Bw%40mail.gmail.com
In-Reply-To: <0E3686EB9FA8AA409AFA0A25468DCE4301309C3DE865@BOM-VMBX-HO.bom.gov.au>

```
On Mon, Aug 16, 2010 at 00:53, David Scurrah <D.Scurrah@bom.gov.au> wrote:

Thanks for not sending a classified patch.

Have you seen the "[bash-completion] syntax error near unexpected
token '<' on line 123 in bash_completion.d/git" thread recently on
this list? It probably describes the problem you're having.

```

## David Scurrah, 2010-08-16 05:45

Subject: Re: Patch for git-completion.bash [SEC=UNCLASSIFIED]
Message-ID: <0E3686EB9FA8AA409AFA0A25468DCE4301309C3DE86C@BOM-VMBX-HO.bom.gov.au>
URL: https://gitlist.dev/e/0E3686EB9FA8AA409AFA0A25468DCE4301309C3DE86C%40BOM-VMBX-HO.bom.gov.au
In-Reply-To: <AANLkTi=QY+3M9jpuwXVcH1-yFhNQrjTfL9epTL-OYGMY@mail.gmail.com>

```
Thanks reading the list showed running as sh instead of bash was my problem

I am trying to combine the very useful git-prompt
(http://volnitsky.com/project/git-prompt/) and git-completion.bash
scripts as each had something the other lacked.

The problem is that while git-prompt suggests you start it from .bashrc
via 
[[ $- == *i* ]]   &&   . /path/to/git-prompt.sh

Using . to execute git-completion.bash runs sh which runs bash in sh
emulation mode which makes it fail in several places.

I did see a thread about an attempt to make the script zsh compliant as
well, I wonder if it would be better to just make it sh compliant so
that it is more portable, I guess that's too hard

regards

David Scurrah
> >
> > On Mon, 2010-08-16 at 04:14 +0000, Ævar Arnfjörð Bjarmason wrote:
> >> On Mon, Aug 16, 2010 at 04:09, David Scurrah <D.Scurrah@bom.gov.au> wrote:
> >> > Hi,
> >> >
> >> > On Mon, 2010-08-16 at 01:09 +0000, Ævar Arnfjörð Bjarmason wrote:
> >> >> On Mon, Aug 16, 2010 at 00:53, David Scurrah <D.Scurrah@bom.gov.au> wrote:
> >> >>
> >> >> Thanks for not sending a classified patch.
> >> >
> >> > Ha ha - the Australian Government has to have email classifications for
> >> > every email sent - classified I believe just means I do not wish you to
> >> > share it not that it is a state secret (see one implementation
> >> > http://www.janus.net.au/janusSEAL/Outlook/features)
> >> >
> >> >
> >> >> Have you seen the "[bash-completion] syntax error near unexpected
> >> >> token '<' on line 123 in bash_completion.d/git" thread recently on
> >> >> this list? It probably describes the problem you're having.
> >> >
> >> > I'll looked hard and can't find the thread you mentioned
> >> > and the current version on my Fedora system does not complete first
> >> > level commands like checkout ...
> >>
> >> http://comments.gmane.org/gmane.comp.version-control.git/151723
> >>
> >> > Maybe there is something I am missing.
> >>
> >> You're probably executing the completion file in bash's sh compatibility mode.



```
