Fix temp file handling

[rasjani]

Changelog: Bugfix: Fix check_output_runner() to handle dirs with whitespaces.
Docs: Omit

Close #6694

• Refer to the issue that supports this Pull Request.
• If the issue has missing info, explain the purpose/use case/pain/need that covers this Pull Request.
• I've read the Contributing guide.
• I've followed the PEP8 style guides for Python code.
• I've opened another PR in the Conan docs repo to the develop branch, documenting this one.

If TEMP/TMPDIR envs point to a directory that contains whitespaces, check_output_runner() fails. Generated filename needs to be quoted for it to be passed in into shell properly.

[memsharded]

This fix looks OK.
It would be great to have a test that covers this case. Could you please try to add it? If you need help ask, we can give some guidance, or we can contribute the test ourselves.

Thanks for contributing this!

[rasjani]

Shouldn’t be an issue if there’s already existing tests.

[memsharded]

There is the unix_to_dos_unit_test in conans\test\unittests\util\tools_test.py, it is a bit of a mix test file, but feel free to add it there.

[rasjani]

test added. verified on osx and fix works there at least as intended

[memsharded]

Excellent, many thanks!