Winmerge can't support paths in command line paramters with spaces in quotes.
winmerge "dir/d ir1/" dir/dir2
Logged In: YES
Can you please tell at least which version of WinMerge you have?
Recent versions should be ok with command line handling. Though you should use "\" as path separator in Windows...
Logged In: YES
Winmerge version 188.8.131.52 Unicode
OS: WinXP Prof
I want to compare two directories
if directory path had no spaces - no problem.
Command line: winmerge D:\temp\test\dir2\ D:\temp\test\dir2\
command line: winmerge "D:\temp\test\dir 1\" D:\temp\test\dir2\
winmerge can't recognize cmd arguments in this case.
Please check attached screenshot.
File Added: winmerge.png
Thanks for the information.
I briefly tested some paths with spaces in them. But couldn't see any problem.
Then I created exactly same paths you have below (thanks!). If I write the paths like this:
winmerge "D:\temp\test\dir 1" D:\temp\test\dir2
there is no problem, but if I write the paths like this:
winmerge "D:\temp\test\dir 1\" D:\temp\test\dir2\
WinMerge behaves like you describe.
So the problem is in handling the last "\" in the path.
Changing the summary, as this is not about spaces but firs path ending with \. That seems to confuse MFC library's parser.
Original and altered file
Ok, I have a patch for this bug. It is a workaround for bugs in MFC/Windows in handling command line parameters. It checks if there is parenthesis inside path WinMerge gets. If there is, it means there is two paths instead of one. Then we just split the path to two paths.
File Added: FixCmdLine.zip
I've committed my fix to SVN trunk:
Completed: At revision: 5347
So the next experimental will have this bug fixed, and hopefully next stable release too (if no other problems are found).
Gah. Of course I meant quotation marks, not parentheses! Updating summary again.
There was much older similar bug report I just closed:
BUG: #1565867 WinMerge doesn't accept trailing "\" in command line paths
Committed the fix to 2.8 branch:
Completed: At revision: 5378
The fix will be a part of 2.8.2 release.
Log in to post a comment.