add missing quote around set arguments in bat

[dirk-thomas]

No description provided.

[codecov-io]

Codecov Report

Merging #279 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #279   +/-   ##
=======================================
  Coverage   79.94%   79.94%           
=======================================
  Files          54       54           
  Lines        3141     3141           
  Branches      521      521           
=======================================
  Hits         2511     2511           
  Misses        589      589           
  Partials       41       41

Impacted Files	Coverage Δ
colcon_core/shell/bat.py	93.44% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a06f55...5d1af57. Read the comment docs.

[hidmic]

Not just yet, getting:

if "%PYTHONPATH:~-1%"==";" set "PYTHONPATH=%PYTHONPATH:~0,-1%"
'if' is not recognized as an internal or external command,
operable program or batch file.

with these changes.

[dirk-thomas]

@hidmic Please try again with the updated patch.

[jacobperron]

Approval under the assumption this fixes things (I haven't given it a try yet).

[dirk-thomas]

I haven't given it a try yet.

I can wait for that - for this part it's better to double/triple check 😉

[jacobperron]

Works for me. Testing in a small workspace of a few packages, it sets and strips the semi-colcon from CMAKE_PREFIX_PATH and AMENT_PREFIX_PATH correctly.

[hidmic]

Took me a while to get back to this, but I can confirm the patch works just fine. Thanks!