comby icon indicating copy to clipboard operation
comby copied to clipboard

`comby -review` is broken for filepaths containing spaces

Open lehmacdj opened this issue 3 years ago • 1 comments

Describe the bug When running comby -review to make a change to files where their filepath contains spaces after accepting a change comby fails with the error:

[debug] File to patch: No file found--skip this patch? [y] 1 out of 1 hunks ignored--saving rejects to Oops.rej
,
Error attempting patch, command exited with 1.
Run the command again with DEBUG_COMBY=1 set in the environment for more info.
Press any key to continue, or exit now (Ctrl-C).

Reproducing Please see this repository that contains a reproduction of the bug, and detailed instructions for reproducing it: https://github.com/lehmacdj/comby-bug-spaces-in-filename-broken

Expected behavior After accepting the change, the change should be applied exactly as though comby -in-place were used instead.

Additional context I am using version 1.8.1 of comby which is the latest release

lehmacdj avatar Jan 20 '23 22:01 lehmacdj

Thanks for the clear reproduction instructions ❤️ . I'll look into this

rvantonder avatar Feb 22 '23 04:02 rvantonder