tool executables with spaces cannnot be quoted on windows

[alexsconway]

In my project.yaml I have

:tools:
  :test_compiler:
    :executable: C:\Program Files\Microchip\xc8\v2.46\bin\xc8-cc.exe
    :name: 'Microchip XC8 compiler'

this used to work fine in ceedling 0.31.1.
In ceedling 1.01 (ruby3.4, windows 11) I get this error:
ERROR: :tools >> :test_compiler >> :executable => C:/Program does not exist on disk

I looked thru the breaking changes and found I shoudl be escape quoting it, so I tried that:
:executable: \"C:\Program Files\Microchip\xc8\v2.46\bin\xc8-cc.exe\"
resulting in:
ERROR: :tools >> :test_compiler >> :executable => /"C:/Program does not exist on disk

other attempts...
:executable: \"C:\\Program Files\\Microchip\\xc8\\v2.46\\bin\\xc8-cc.exe\"
ERROR: :tools >> :test_compiler >> :executable => /"C://Program does not exist on disk

:executable: \"C:/Program Files/Microchip/xc8/v2.46/bin/xc8-cc.exe\"
ERROR: :tools >> :test_compiler >> :executable => /"C:/Program does not exist on disk

I then created a dummy compiler in the local directory called "Stupid Compiler.exe"
(btw, "Code Cruncher" is not a good example because "Code" invokes VScode!)
:executable: \"Stupid Compiler.exe\"
ERROR: :tools >> :test_compiler >> :executable => /"Stupid does not exist on disk

It looks like all slashes are being coverted to forward slashes, including all backslash-escapes

Only workaround I have atm is to add the Microchip compiler to %PATH%
Thanks
Alex

[Letme]

Based on your test (Stupid does not exist), I would say you also need to escape space character for this to work? This is kinda not a Ceedling problem, but more of a Ruby problem.

:executable: "C:/Program\ Files/Microchip/xc8/v2.46/bin/xc8-cc.exe"

[alexsconway]

I've jusrt tried your suggestion, and the result is
ERROR: :tools >> :test_compiler >> :executable => C:/Program does not exist on disk

It seems that this used to be ok in 0.31.1, due to this revision
https://github.com/ThrowTheSwitch/Ceedling/pull/9/commits/6bb1f8c41eb8305599904f5fffb942ecb6fa572e

but that was reverted in this rev, citing non-specific "problems"
https://github.com/ThrowTheSwitch/Ceedling/commit/ab16cc6fd6a6f03cd071af9a7b382f1cbf7ab1d5

I also see that there is a function used, tool_executor_helper.osify_path_separators(..) that pretty much explicitly replaces all slashes with forward slashes (In Ceedling, not Ruby), therefore it is not possible to escape any character.

[swaldhoer]

Have you tried :executable: "\"C:/Program Files/Microchip/xc8/v2.46/bin/xc8-cc.exe\""?

[alexsconway]

Thank you swaldhoer!
I've just corrected my original post, where I hadn't put code around the code so the various forward and back slashes had disappeared. My quoting, escaping and markdowning is weak obviously.
The Breaking Changes doc says to do this:
:executable: \"Code Cranker\"
but it seems I should be doing this
:executable: "\"Code Cranker\""

[alexsconway]

So this is really a documentation issue then. What should I do to get the Breaking Changes doc corrected?
Should I raise another case, referencing this one or...?
Thanks
Alex