egg icon indicating copy to clipboard operation
egg copied to clipboard

Possible bug with CR in merge filename

Open garyo opened this issue 13 years ago • 2 comments

Unfortunately I don't know how I got into this situation, but today I got egg into a state where it stopped working, because of an extra CR (^M) in one of the .git/rebase-merge/* files. Egg failed to parse the branch name "topic/foo^M" (where ^M was a real CR in the actual file) as a rev.

I wonder if egg-file-as-string should strip CRs, at least on Windows?

garyo avatar Nov 16 '12 18:11 garyo

I think the bug is: egg uses insert-file-contents-literally() instead of insert-file-contents(). We'll have to audit all the occurrences of insert-file-contents-literally() to see if it makes sense or not to convert them to insert-file-contents().

bogolisk avatar Nov 18 '12 22:11 bogolisk

I've just looked through the code and realized that all the calls to insert-file-contents-literally should be changed to insert-file-contents.

bogolisk avatar Nov 18 '12 22:11 bogolisk