Início / unix / Perguntas / 768806
Accepted
wyc
Asked: 2024-02-08 14:23:36 +0800 CST

Como adicionar um sinalizador de correspondência exata à seguinte função git + sed?

  • 772

Quero criar uma função bash onde você possa localizar e substituir sed enquanto ignora os arquivos e pastas em .gitignore. Também quero poder adicionar sinalizadores git grep ao comando. Por exemplo, para uma correspondência exata, devo poder adicionar -w.

Isto é o que tenho até agora:

gs {
  local grep_options=()
  local search_pattern
  local replacement

  while [[ $1 =~ ^- ]]; do
    grep_options+=("$1")
    shift
  done

  search_pattern=$1
  replacement=$2

  git grep -l "${grep_options[@]}" "$search_pattern" | xargs sed -i "s/$search_pattern/$replacement/g"
}

gs pattern replacementfará a pesquisa e substituição com sucesso. Mas gs -w pattern replacementnão fará nada.

Por que isso acontece e como consertar?

bash

1 respostas

  1. Best Answer

    Você tem um erro de sintaxe na declaração da função.

    Em vez de

    w { ... }

    você precisa:

    w() { ... }

    Sempre passe seus scripts para https://shellcheck.net antes de pedir ajuda:

    $ shellcheck file

In file line 1:
gs {
^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.
   ^-- SC1083 (warning): This { is literal. Check expression (missing ;/\n?) or quote it.


In file line 2:
  local grep_options=()
  ^---^ SC2168 (error): 'local' is only valid in functions.


In file line 3:
  local search_pattern
  ^---^ SC2168 (error): 'local' is only valid in functions.


In file line 4:
  local replacement
  ^---^ SC2168 (error): 'local' is only valid in functions.


In file line 15:
}
^-- SC1089 (error): Parsing stopped here. Is this keyword correctly matched up?

For more information:
  https://www.shellcheck.net/wiki/SC2148 -- Tips depend on target shell and y...
  https://www.shellcheck.net/wiki/SC2168 -- 'local' is only valid in functions.
  https://www.shellcheck.net/wiki/SC1083 -- This { is literal. Check expressi...
    • 2

relate perguntas