[Bash-completion-devel] Bug#734095: bash-completion: Please replace 'grep' with 'command grep' within several functions.

2014-03-21 Thread David Paleino
tags 734095 upstream fixed-upstream
thanks

On Fri, 21 Mar 2014 22:27:22 +0100, Tomasz Nowiński wrote:

 Hi,
 
 This bugs came back in version 2.1-3!

It never disappeared in 2.1-2.

Simply, the bug has been fixed upstream, while not in the Debian package...
yet. :)

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174

___
Bash-completion-devel mailing list
Bash-completion-devel@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/bash-completion-devel

[Bash-completion-devel] Bug#734095: bash-completion: Please replace 'grep' with 'command grep' within several functions.

2014-01-03 Thread Tomasz Nowiński
Package: bash-completion
Version: 1:2.1-2
Severity: normal

Hi,

In my shell I have defined a custom grep function that overrides the original
command. If I try to use make in directory without Makefile or with some
makefiles, i get error from grep grep: conflicting matchers specified,
because my grep uses -E.

Replacement 'grep' with 'command grep'

144c144
 local reset=$( set +o | grep -F posix ); set +o posix # for (...)
---
 local reset=$( set +o | command grep -F posix ); set +o posix # for
(...)

resolves the problem.

This is not only example of using grep not directly from it's path:

 $ grep '\grep '  /usr/share/bash-completion/completions/*| grep -v '(command\
|\.)grep'
/usr/share/bash-completion/completions/c++:cc  --version 2/dev/null | grep
-q GCC  complete -F _gcc cc  || :
/usr/share/bash-completion/completions/c++:c++ --version 2/dev/null | grep
-q GCC  complete -F _gcc c++ || :
/usr/share/bash-completion/completions/cc:cc  --version 2/dev/null | grep
-q GCC  complete -F _gcc cc  || :
/usr/share/bash-completion/completions/cc:c++ --version 2/dev/null | grep
-q GCC  complete -F _gcc c++ || :
/usr/share/bash-completion/completions/colormake:local reset=$( set +o
| grep -F posix ); set +o posix # for (...)
/usr/share/bash-completion/completions/dict:# it down with grep if $cur
looks like something that's safe to embed
/usr/share/bash-completion/completions/g++:cc  --version 2/dev/null | grep
-q GCC  complete -F _gcc cc  || :
/usr/share/bash-completion/completions/g++:c++ --version 2/dev/null | grep
-q GCC  complete -F _gcc c++ || :
/usr/share/bash-completion/completions/g77:cc  --version 2/dev/null | grep
-q GCC  complete -F _gcc cc  || :
/usr/share/bash-completion/completions/g77:c++ --version 2/dev/null | grep
-q GCC  complete -F _gcc c++ || :
/usr/share/bash-completion/completions/gcc:cc  --version 2/dev/null | grep
-q GCC  complete -F _gcc cc  || :
/usr/share/bash-completion/completions/gcc:c++ --version 2/dev/null | grep
-q GCC  complete -F _gcc c++ || :
/usr/share/bash-completion/completions/gcj:cc  --version 2/dev/null | grep
-q GCC  complete -F _gcc cc  || :
/usr/share/bash-completion/completions/gcj:c++ --version 2/dev/null | grep
-q GCC  complete -F _gcc c++ || :
/usr/share/bash-completion/completions/gmake:local reset=$( set +o |
grep -F posix ); set +o posix # for (...)
/usr/share/bash-completion/completions/gnumake:local reset=$( set +o |
grep -F posix ); set +o posix # for (...)
/usr/share/bash-completion/completions/gpc:cc  --version 2/dev/null | grep
-q GCC  complete -F _gcc cc  || :
/usr/share/bash-completion/completions/gpc:c++ --version 2/dev/null | grep
-q GCC  complete -F _gcc c++ || :
/usr/share/bash-completion/completions/lintian:tags=$( grep -e ^Tag
/usr/share/lintian/checks/*.desc | cut -d\  -f2 )
/usr/share/bash-completion/completions/lintian:match=$(grep -nE
^Tag: $item$ /usr/share/lintian/checks/*.desc \
/usr/share/bash-completion/completions/lintian:checks=$(grep -e ^Check-
Script -e ^Abbrev \
/usr/share/bash-completion/completions/lintian:match=$(grep -nE
^(Check-Script|Abbrev): $item$ \
/usr/share/bash-completion/completions/lintian:todisable=$(grep -e
^Check-Script -e ^Abbrev $match | \
/usr/share/bash-completion/completions/lintian:infos=$(grep -e ^Collector
/usr/share/lintian/collection/*.desc \
/usr/share/bash-completion/completions/lintian:match=$( grep -nE
^Collector: $item$ \
/usr/share/bash-completion/completions/lintian-info:tags=$( grep -e ^Tag
/usr/share/lintian/checks/*.desc | cut -d\  -f2 )
/usr/share/bash-completion/completions/lintian-info:match=$(grep
-nE ^Tag: $item$ /usr/share/lintian/checks/*.desc \
/usr/share/bash-completion/completions/lintian-info:checks=$(grep -e
^Check-Script -e ^Abbrev \
/usr/share/bash-completion/completions/lintian-info:match=$(grep
-nE ^(Check-Script|Abbrev): $item$ \
/usr/share/bash-completion/completions/lintian-info:
todisable=$(grep -e ^Check-Script -e ^Abbrev $match | \
/usr/share/bash-completion/completions/lintian-info:infos=$(grep -e
^Collector /usr/share/lintian/collection/*.desc \
/usr/share/bash-completion/completions/lintian-info:match=$( grep
-nE ^Collector: $item$ \
/usr/share/bash-completion/completions/make:local reset=$( set +o |
grep -F posix ); set +o posix # for (...)
/usr/share/bash-completion/completions/pkgadd:
pkginst_list=$(strings $(dequote $device) | grep ^PKG= | sort -u | cut -d=
-f2)
/usr/share/bash-completion/completions/pmake:local reset=$( set +o |
grep -F posix ); set +o posix # for (...)
/usr/share/bash-completion/completions/rdict:# it down with grep if
$cur looks like something that's safe to embed
/usr/share/bash-completion/completions/slackpkg:# options list is based on
`grep '\-.*\=.*)' /usr/sbin/slackpkg | cut -f1 -d\)`