Make it faster using builtins

[michaeldfallen]

Make it faster using builtins

[sjaveed]

Nice! I've been using git symbolic-ref in my own PS1 which is a lot dumber than git-radar so this should come in handy for me. Thanks @jart!

Edit: bonus points for the ï in naïve :-)

[zx8]

Disclaimer: I'm not a git-radar user nor do I have problems with the responsiveness of my prompt, I was just browsing the project's issues out of curiosity after following a link from a friend!

Anyway, this bit needs to be revised, because / is a valid character in a branch name:

jartps1_gitbranch="${head##*/}"

Example:

$ head=release/1.0
$ echo "${head##*/}"
1.0

[jart]

@zx8 Wow thanks for finding that bug. I updated the code in my original comment. PTAL?

I'm not 100% sure what the best approach is, since I don't know everything that could be contained in that file. So I tried to pick the safest heuristic possible.

[zx8]

Hmm, think you've overcomplicated it a bit, how about something simple like this:

$ h=refs/heads/release/1/2/3
$ echo ${h##refs/heads/}
release/1/2/3

[jart]

@zx8 The ## syntax wouldn't work since the HEAD file contains ref: refs/heads/branch on my end. I guess I could just say ${h##ref: refs/heads/} though. But that would be more fragile without the simpler fallback. I imagine that HEAD file can contain all sorts of things. Like if you checkout a SHA1 ID, it won't have the ref:blahblah prefix; it'll just contain the hash.