Partial filename display when space in it

[rockandska]

Original

diff --git a/a b b/a b
new file mode 100644
index 0000000..e69de29

So-Fancy:

─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
added: a
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

The filename is "a b" and not "a"

$ diff-so-fancy --version
Diff-so-fancy: https://github.com/so-fancy/diff-so-fancy
Version      : 1.2.6

Regards,

[scottchiefbaker]

Is this true of all files or just files that start with a or b? A/B are special cases in code, so it may be isolated just to them.

[rockandska]

Seem the same for 'c d'

$ git diff /dev/null c\ d
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
added: c
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

And the same for another test

git diff /dev/null more\ text
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
added: more
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

[scottchiefbaker]

I'm not able to recreate what you're seeing:

mkdir baker
cd baker
git init
echo 'Line1' > 'foo bar.txt'
git add .
git commit -m '1'
echo 'Line1a' > 'foo bar.txt'
git diff

This results in:

───
modified: foo bar.txt
───
@ foo bar.txt:1 @

[rockandska]

Because you are not in the same state as me.
I'm in the case where the file is not already committed (untracked) so the diff is done against /dev/null

$ git diff -- /dev/null c\ d 
diff --git a/c d b/c d
new file mode 100644
index 0000000..e69de29
$ git diff -- /dev/null c\ d | diff-so-fancy 
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
added: c
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

and it works as it does when there is no space in the filename

$ git diff -- /dev/null test-file
diff --git a/test-file b/test-file
new file mode 100644
index 0000000..e69de29
$ git diff -- /dev/null test-file | diff-so-fancy 
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
added: test-file
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

[scottchiefbaker]

Still not tracking... when I create a new untracked file named a b and run git diff on it, I get correct output:

:git diff -- /dev/null a\ b
────────────────────────────────────
added: a b
────────────────────────────────────
@ a b:1 @

Can you give me a full set of instructions to recreate the issue?

[rockandska]

Don't add content to the file is the part who let appear this behavior

$ touch $'un test'
$ git diff -- /dev/null un\ test | diff-so-fancy 
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
added: un
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
$ echo "test" > $'un test'
$ git diff -- /dev/null un\ test | diff-so-fancy 
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
added: un test
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
@ un test:1 @
test
$ 

[scottchiefbaker]

OK I can recreate it now. Can you try the fix in 24f8924 please?

[rockandska]

LGTM

The filename is no truncated anymore when the file is empty and the filename contain a space with 24f8924

[rockandska]

Remaining on thing, the cleanup of the color doesn't seem to be done in this case.

[scottchiefbaker]

I'm unable to recreate the issue you see above. Can you confirm that follow does/doesn't show color on the following line.

git diff -- /dev/null a\ b; echo "This is another line of text"

On my test the subsequent line of text is in the neutral gray color I would expect.

[rockandska]

Seems to happen only when pipe is used, and not related to the fix

[scottchiefbaker]

Can you test again, I just landed a fix.