Made bash completion work for OSX

tools/bash-completion

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Flag completion rule for bash.
 # To load in your shell, "source path/to/this/file".
 
-chrome_source=$(readlink -f $(dirname $BASH_SOURCE)/..)
+chrome_source=$(dirname $BASH_SOURCE)/..
+if [ $(uname) = "Darwin" ]
+then
+  chrome_source=$(python -c 'import os,sys;print os.path.realpath(sys.argv[1])'\
+      $chrome_source)

[Evan Martin, 2012/02/17 20:23:17]

Simpler:
chrome_source=$((cd $chrome_source && pwd))

[groby-ooo-7-16, 2012/02/17 22:19:28]

Surely you only meant one level of parens :)

On 2012/02/17 20:23:17, Evan Martin wrote:

+  sed_expr='s/^[^/]*"\([^" /]*\)".*/--\1/p'

[Evan Martin, 2012/02/17 20:23:17]

Why is this different between Darwin and not?

[groby-ooo-7-16, 2012/02/17 22:19:28]

Because Darwin doesn't support \+. And if I'd read ed(1) instead of re_format(7)
right away, I would've known. Ah well - have a cross-platform, if ugly,
alternative w/ latest.

On 2012/02/17 20:23:17, Evan Martin wrote:

+else
+  chrome_source=$(readlink -f $chrome_source)
+  sed_expr='s/^[^/]*"\([^" /]\+\)".*/--\1/p'
+fi
 
 _chrome_flag() {
     local cur targets
     cur="${COMP_WORDS[COMP_CWORD]}"
     targets=$(cd $chrome_source; \
         git ls-files '*switches*' | \
-        xargs sed -ne 's/^[^/]*"\([^" /]\+\)".*/--\1/p')
+        xargs sed -ne "$sed_expr")
     COMPREPLY=($(compgen -W "$targets" -- "$cur"))
     return 0
 }
 
 complete -F _chrome_flag google-chrome
 complete -F _chrome_flag chrome
+if [ $(uname) = "Darwin" ]
+then
+  complete -F _chrome_flag Chromium
+fi