[gitstatus.py] Use --porcelain to make faster in using python

[wkentaro]

Get files' status by parsing output of git status --porcelain to make it faster.

[mcornella]

Hi @wkentaro, I'm using your modification in ohmyzsh/ohmyzsh#4205 because it is the best one here.
You could simplify further by using git status --porcelain -b which shows the remote branch info (including ahead and behind commits):

$ git status --porcelain -b
## fix-git-prompt-plugin...origin/fix-git-prompt-plugin [ahead 2, behind 2]

[wkentaro]

Ok, thanks.

[wkentaro]

I sent PR: mcornella/ohmyzsh#2

[mcornella]

@olivierverdier I can confirm this works perfectly, and it only uses one call to git instead of 4-5 of before.
👍

[mcornella]

I had to push a patch to make this work for UTF-8 locales, see mcornella/ohmyzsh@b289beb and ohmyzsh/ohmyzsh#4205 (comment).

[wkentaro]

Thanks. I updated.

[mcornella]

There are some other caveats with the git status --porcelain -b command. See ohmyzsh/ohmyzsh#4205 (comment)

[wkentaro]

Thanks, I updated.

[wkentaro]

I think this PR should be merged after ohmyzsh/ohmyzsh#4205 is merged.
So plz wait for it.

After it is merged, I will update all same commits to this PR.

[wkentaro]

ohmyzsh/ohmyzsh#4205
is merged and updated the commits. Please review and merge.

[wkentaro]

@olivierverdier
I added test suite on travis.yml. It tests

• output of gitstatus.py
• speed of gitstatus.py

it compares the speed with gitstatus.py at 9f0da67.

[wkentaro]

ping

[olivierverdier]

Thanks for the update. I wrote some comments in the commit. Basically, I think that we need unit tests as well.

[wkentaro]

Ok so you mean we should split the code in gitstatus.py and test each functions?

[olivierverdier]

Yes, exactly. (sorry for the late answer)