fix: when grep is alias (like alias grep='grep --color -Hn'), completions fails;

also, don't invoke sed twice, when one time can be done (sed -e .. -e .. ...)

Author: glhez

bash_completion.bash

@@ -92,9 +92,9 @@ __pom_hierarchy()
 {
     local pom=`_realpath "pom.xml"`
     POM_HIERARCHY+=("$pom")
-    while [ -n "$pom" ] && grep -q "<parent>" "$pom"; do
+    while [ -n "$pom" ] && \grep -q "<parent>" "$pom"; do
 	    ## look for a new relativePath for parent pom.xml
-        local parent_pom_relative=`grep -e "<relativePath>.*</relativePath>" "$pom" | sed 's/.*<relativePath>//' | sed 's/<\/relativePath>.*//g'`
+      local parent_pom_relative=`\grep -e "<relativePath>.*</relativePath>" "$pom" | sed -e 's/.*<relativePath>//' -e 's/<\/relativePath>.*//g'`
 
     	## <parent> is present but not defined, assume ../pom.xml
     	if [ -z "$parent_pom_relative" ]; then
@@ -103,7 +103,7 @@ __pom_hierarchy()
 
     	## if pom exists continue else break
     	parent_pom=`_realpath "${pom%/*}/$parent_pom_relative"`
-        if [ -n "$parent_pom" ]; then 
+      if [ -n "$parent_pom" ]; then 
             pom=$parent_pom
     	else 
     	    break
@@ -187,16 +187,16 @@ _mvn()
     local plugin_goals_wildfly="wildfly:add-resource|wildfly:deploy|wildfly:deploy-only|wildfly:deploy-artifact|wildfly:redeploy|wildfly:redeploy-only|wildfly:undeploy|wildfly:undeploy-artifact|wildfly:run|wildfly:start|wildfly:shutdown|wildfly:execute-commands"
 
     ## some plugin (like jboss-as) has '-' which is not allowed in shell var name, to use '_' then replace
-    local common_plugins=`compgen -v | grep "^plugin_goals_.*" | sed 's/plugin_goals_//g' | tr '_' '-' | tr '\n' '|'`
+    local common_plugins=`compgen -v | \grep "^plugin_goals_.*" | sed -e 's/plugin_goals_//g' -e 's/_/-/g' | tr '\n' '|'`
 
     local options="-Dmaven.test.skip=true|-DskipTests|-DskipITs|-Dtest|-Dit.test|-DfailIfNoTests|-Dmaven.surefire.debug|-DenableCiProfile|-Dpmd.skip=true|-Dcheckstyle.skip=true|-Dtycho.mode=maven|-Dmaven.javadoc.skip=true|-Dgwt.compiler.skip|-Dcobertura.skip=true|-Dfindbugs.skip=true||-DperformRelease=true|-Dgpg.skip=true|-DforkCount"
 
-    local profile_settings=`[ -e ~/.m2/settings.xml ] && grep -e "<profile>" -A 1 ~/.m2/settings.xml | grep -e "<id>.*</id>" | sed 's/.*<id>//' | sed 's/<\/id>.*//g' | tr '\n' '|' `
+    local profile_settings=`[ -e ~/.m2/settings.xml ] && \grep -e "<profile>" -A 1 ~/.m2/settings.xml | \grep -e "<id>.*</id>" | sed -e 's/.*<id>//' -e 's/<\/id>.*//g' | tr '\n' '|' `
 
     local profiles="${profile_settings}|"
     for item in ${POM_HIERARCHY[*]}
     do
-        local profile_pom=`[ -e $item ] && grep -e "<profile>" -A 1 $item | grep -e "<id>.*</id>" | sed 's/.*<id>//' | sed 's/<\/id>.*//g' | tr '\n' '|' `
+        local profile_pom=`[ -e $item ] && \grep -e "<profile>" -A 1 $item | \grep -e "<id>.*</id>" | sed -e 's/.*<id>//' -e 's/<\/id>.*//g' | tr '\n' '|' `
         local profiles="${profiles}|${profile_pom}"
     done
 
@@ -239,9 +239,9 @@ _mvn()
         done
 
     else
-        if echo "${common_lifecycle_phases}" | tr '|' '\n' | grep -q -e "^${cur}" ; then
+        if echo "${common_lifecycle_phases}" | tr '|' '\n' | \grep -q -e "^${cur}" ; then
           COMPREPLY=( $(compgen -S ' ' -W "${common_lifecycle_phases}" -- ${cur}) )
-        elif echo "${common_plugins}" | tr '|' '\n' | grep -q -e "^${cur}"; then
+        elif echo "${common_plugins}" | tr '|' '\n' | \grep -q -e "^${cur}"; then
           COMPREPLY=( $(compgen -S ':' -W "${common_plugins}" -- ${cur}) )
         fi
     fi