pbpy: add a run_with_stdin() wrapper and join command list for all OSes

ProjectBorealis / PBSync

System for syncing Unreal Engine project git repositories

MIT License

79 stars 8 forks source link

Closed commodo closed 3 years ago

commodo commented 3 years ago

This is similar to commit ca91f0a ("pbpy/pbtools.py: join the command list in the run() helpers").

Because the 'shell=True' argument is set the commands don't work on Linux if passed as a Python list.

To address this, we add a run_with_stdin() wrapper that handles the conversion in the same way, and takes a parameter that is the input from stdin.

Signed-off-by: Alexandru Ardelean ardeleanalex@gmail.com

mastercoms commented 3 years ago

Hi, the commands have to be a list on Windows.

commodo commented 3 years ago

Hi, the commands have to be a list on Windows.

ok; will remove the last commit; i wasn't sure about that; that's why i made it a separate commit

commodo commented 3 years ago

Changelog v1 -> v2:

remove commit pbpy: add a run_with_stdin() wrapper for calling with input from stdin