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

Closed
opened 1 year ago by tukusejssirs · 3 comments

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 commented 1 year ago
Poster

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 commented 1 year ago
Owner

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 commented 1 year ago
Poster

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. :)
Sign in to join this conversation.
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.