Skip to content

Correctly parse output of git status -z#76

Merged
WtfJoke merged 2 commits intortcTo:developfrom
ohumbel:issue75
Nov 6, 2015
Merged

Correctly parse output of git status -z#76
WtfJoke merged 2 commits intortcTo:developfrom
ohumbel:issue75

Conversation

@ohumbel
Copy link
Member

@ohumbel ohumbel commented Nov 3, 2015

It was wrong to strip the lines, because they might start with a blank.
Fixes #75.

Just for the record: We use the date of the first commit to this project.
This should remove many fatal error messages in the test output, e.g. on Travis.
As a side effect, this reveals the problem stripping ' D ' lines of git status -z.
Enables recognition of deletion lines like
[ D aFolder/.jazzignore]
Fixes rtcTo#75.
WtfJoke added a commit that referenced this pull request Nov 6, 2015
Correctly parse output of git status -z
@WtfJoke WtfJoke merged commit 1ee090d into rtcTo:develop Nov 6, 2015
@ohumbel ohumbel deleted the issue75 branch November 13, 2015 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants