gitinspector --metrics errs out on paths with spaces

[ejwa]

gitinspector --metrics errs out on paths with spaces

[GoogleCodeExporter]

Hi. Gitinspector will be able to handle multiple repositories at some point 
(and group them together). What you are seeing doesn't really relate to 
gitinspector - it's related to the terminal. If you want to pass something as 
one path when invoking gitinspector from the terminal, just do the following:

gitinspector.py --metrics "/other files/go_here"

Also, This patch would break terminals that don't use '"' to group arguments 
together.

/Adam Waldenberg

Original comment by [email protected] on 19 Nov 2014 at 6:19

• Changed state: Invalid

[GoogleCodeExporter]

I agree the problem is with the terminal.
However, it's not the /path/to/repo that contains spaces.
It's files inside that repo (that have spaces in their filename) that break 
gitinspector.py

Therefore I think it's a very valid bug :-)

Original comment by [email protected] on 20 Nov 2014 at 8:11

[GoogleCodeExporter]

Aha. That makes more sense... Sorry.

I'm actually doing something similar in blame.py and it seems to be working for 
most users (and terminals ;)).

/Adam Waldeneberg

Original comment by [email protected] on 21 Nov 2014 at 1:45

• Changed state: Started
• Added labels: Priority-Critical
• Removed labels: Priority-Medium

[GoogleCodeExporter]

This issue was closed by revision dd50994f2c4c.

Original comment by [email protected] on 21 Nov 2014 at 1:56

• Changed state: Fixed

[GoogleCodeExporter]

Issue 56 has been merged into this issue.

Original comment by [email protected] on 9 Feb 2015 at 4:53