search

Dushistov / sdcv

https://dushistov.github.io/sdcv/
GNU General Public License v2.0
294 stars 42 forks source link

Pass parameters individually to the test #35

Closed nijel closed 6 years ago

nijel commented 6 years ago

This way we can properly quote path.

This is probably better approach than just removing the quotes as done in #34.

Dushistov commented 6 years ago

I checked on my machine with bash 4.4.12 and for both #34 and #35 t_json test failed, with: "\\nresult of test" vs "\nresult", may be instead of quotes use something like this or python?

nijel commented 6 years ago

Apparently there is something fishy in the way how shells handle \n escape sequence. I've done some more testing and https://github.com/Dushistov/sdcv/pull/37 works in all shells I have around. Can you try if that fixes problem for you as well?

Frenzie commented 6 years ago

Did you try explicitly escaping \n like \\n (or perhaps rather \\\\n to add an extra degree of escaping to everything) instead of depending on fallback behavior?