History Of Ticket 8d073be880
Not logged in
HomeFilesLeavesTimelineBranchesTagsTicketsWikiLogin
Status Timeline
Ticket change [a24ed38e9a] (rid 5208) by drh on 2009-09-14 19:50:27:
  1. Appended to comment:

    drh added on 2009-09-14 19:50:27:
    I have retested and I am unable to reproduce the problem. I'm setting gdiff-command to

    C:\Program Files\WinMerge\WinMergeU.exe

    And then I run "gdiff" from both a cygwin shell and a regular DOS box and it works great in both cases.

Ticket change [93ca33fd8a] (rid 5204) by anonymous on 2009-09-14 18:40:57:
  1. Appended to comment:

    anonymous claiming to be alison.rowland@filollc.com added on 2009-09-14 18:40:57:
    This is still broken under Cygwin, with the latest Fossil version (eb1db585a5 2009-09-14 16:56:29 UTC).

Ticket change [85c2f3f6a2] (rid 5141) by anonymous on 2009-09-11 01:32:34:
  1. Appended to comment:

    anonymous added on 2009-09-11 01:32:34:
    drh - i noticed in checkin aeaee1f38512181e that you quoted the entire command for windows, including parameters. i don't think that is what you need. for example, your patch would do this for gdiff-command:

    "C:\Program Files\Beyond Compare 3\BComp.exe file1.cpp file2.cpp"
    the above would still result in the OS thinking that C:\Program was the command you were executing and Files\Beyond etc. is a space delimited parameter list. for commands with spaces in them, you need to quote the command and then quote any parameters with spaces in them. to be safe, you can just quote all parameters separately, so what you want is one of these:
    "C:\Program Files\Beyond Compare 3\BComp.exe" file1.cpp file2.cpp
"C:\Program Files\Beyond Compare 3\BComp.exe" "file1.cpp" "file2.cpp"

Ticket change [ad62476e92] (rid 5140) by drh on 2009-09-10 23:01:15:
  1. Appended to comment:

    drh added on 2009-09-10 23:01:15:
    Fixed by check-in aeaee1f38512181e.

  2. Change resolution to "Fixed"
  3. Change status to "Fixed"
Ticket change [63e2b019d4] (rid 5134) by anonymous on 2009-09-10 21:24:08:
  1. Change comment to "for the diff-command, gdiff-command, and editor settings, assigning a path with spaces causes failures when fossil attempts to launch the external tools. i even tried to assign the settings with quotes but they are (probably) stripped by the OS on the command line. this was observed using windows xp and powershell.<br> <br> RW<br> <pre>PS C:\svn\rpw\Zyrain\Test> fossil setting gdiff-command "C:\Program Files\Beyond Compare 3\BComp.exe" PS C:\svn\rpw\Zyrain\Test> fossil settings autosync (local) 1 diff-command editor (local) C:\Program Files\JGsoft\EditPadPro6\EditPadPro.exe gdiff-command (local) C:\Program Files\Beyond Compare 3\BComp.exe http-port localauth (local) 0 clearsign (local) off pgp-command mtime-changes proxy web-browser PS C:\svn\rpw\Zyrain\Test> fossil gdiff sct.txt 'C:\Program' is not recognized as an internal or external command, operable program or batch file. PS C:\svn\rpw\Zyrain\Test> fossil ver This is fossil version [713b8be852] 2009-08-28 22:59:27 UTC </pre>"
  2. Change foundin to "713b8be852"
  3. Change private_contact to "2359089ef753bc96e12743e299243d5232c749fa"
  4. Change severity to "Important"
  5. Change status to "Open"
  6. Change title to "diff/editor/gdiff settings don't support paths with spaces"
  7. Change type to "Code_Defect"

Fossil version [c0211010a1] 2009-12-13 01:31:36