#6 `subbash/prompt`: GBranch: simplify the command

已關閉
tukusejssirs5 年之前創建 · 3 條評論

I was looking for a command to get the current branch name and I found this (src):

git rev-parse --abbrev-ref HEAD

Would it not be better than the current method?

# Current method
git status --porcelain=2 -b 2>/dev/null | tr '\n' ':' | \
awk 'match($0,/# branch.head [^ :]+/) {print substr($0,RSTART+14,RLENGTH-14)}'
I was looking for a command to get the current branch name and I found this ([src](https://stackoverflow.com/a/12142066/3408342)): ```bash git rev-parse --abbrev-ref HEAD ``` Would it not be better than the current method? ```bash # Current method git status --porcelain=2 -b 2>/dev/null | tr '\n' ':' | \ awk 'match($0,/# branch.head [^ :]+/) {print substr($0,RSTART+14,RLENGTH-14)}' ```
tukusejssirs 評論 5 年之前'
發佈者

If anything, it should faster (I’ve removed the branch name from the output):

$ time git status --porcelain=2 -b 2>/dev/null | tr '\n' ':' | \
awk 'match($0,/# branch.head [^ :]+/) {print substr($0,RSTART+14,RLENGTH-14)}'

real    0m0.008s
user    0m0.007s
sys 0m0.007s

$ time git rev-parse --abbrev-ref HEAD

real    0m0.003s
user    0m0.003s
sys 0m0.000s
If anything, it should faster (I’ve removed the branch name from the output): ```bash $ time git status --porcelain=2 -b 2>/dev/null | tr '\n' ':' | \ awk 'match($0,/# branch.head [^ :]+/) {print substr($0,RSTART+14,RLENGTH-14)}' real 0m0.008s user 0m0.007s sys 0m0.007s $ time git rev-parse --abbrev-ref HEAD real 0m0.003s user 0m0.003s sys 0m0.000s ```
demure 評論 5 年之前'
所有者

Currently the awk is using a variable.
So a better test would be:

~ -> cd projects/dotfiles/
dotfiles + [M] -> GStatus="$(git status --porcelain=2 -b 2>/dev/null | tr '\n' ':')"
dotfiles [M] -> time echo ${GStatus} | awk 'match($0,/# branch.head [^ :]+/) {print substr($0,RSTART+14,RLENGTH-14)}'
master

real    0m0.007s
user    0m0.007s
sys     0m0.001s
dotfiles [M] -> time git rev-parse --abbrev-ref HEAD
master

real    0m0.005s
user    0m0.000s
sys     0m0.005s
dotfiles [M] ->

I lean a bit more towards keeping the code base uniform as the rest of the git checking also uses the git status (and it seems like minimal difference in time to me.)

Currently the awk is using a variable. So a better test would be: ``` ~ -> cd projects/dotfiles/ dotfiles + [M] -> GStatus="$(git status --porcelain=2 -b 2>/dev/null | tr '\n' ':')" dotfiles [M] -> time echo ${GStatus} | awk 'match($0,/# branch.head [^ :]+/) {print substr($0,RSTART+14,RLENGTH-14)}' master real 0m0.007s user 0m0.007s sys 0m0.001s dotfiles [M] -> time git rev-parse --abbrev-ref HEAD master real 0m0.005s user 0m0.000s sys 0m0.005s dotfiles [M] -> ``` I lean a bit more towards keeping the code base uniform as the rest of the git checking also uses the git status (and it seems like minimal difference in time to me.)
tukusejssirs 評論 5 年之前'
發佈者

No problem. I just gave a suggestion which can be rejected—as always. :)

No problem. I just gave a suggestion which can be rejected—as always. :)
demure5 年之前 關閉
登入 才能加入這對話。
未選擇標籤
bug
未選擇里程碑
未指派成員
2 參與者
正在加載...
取消
保存
尚未有任何內容