filenames with spaces no longer error #7
Conversation
|
This seems to fix the basic operation, but it's not fixing things when you use the -f option. Even so, if passing files with spaces as arguments, you might want to escape the spaces as well. A little sed magic could fix that up. |
|
Hi James. Thanks for the fix! I'm going to have to update the test-repo with some files with spaces in them before I can merge in your pull request. Time is a bit tight for me right not, but if you are interested in doing that please feel welcome. |
Filenames are trimmed, spaces escaped. The output methods were pruned and use awk more. Header row on output added at the end, if needed.
|
I did a bunch more work on it, getting awk in there all over the place. I think I've got it working in all your use cases, but I haven't update the tests to really see. When one of us gets a chance, I think we'll know for sure. |
|
One final note. In your final example: To support spaces, xargs will require the -0 argument. |
Rather than getting column 2, I get column 2 to the end of the line. Spaces in filenames are handled properly.