Re: [PATCH rtems-tools] rtemstoolkit: add support for executing pipe command

2023-06-19 Thread Muhammad Sulthan Mazaya
Sorry, sent the wrong patch, it should be the v2. Ignore this one. On Tue, Jun 20, 2023 at 12:30 PM Muhammad Sulthan Mazaya < msulthanmaz...@gmail.com> wrote: > The previous implementation of execute command on rtems-tools config does > not support command line pipe operation. So something like:

[PATCH rtems-tools] rtemstoolkit: add support for executing pipe command

2023-06-19 Thread Muhammad Sulthan Mazaya
The previous implementation of execute command on rtems-tools config does not support command line pipe operation. So something like: %execute %{command} | %{trim_command} Would not work, since the "| %{trim_command}" part is treated as an additional command option. This patch is intended to

Re: [PATCH rtems-tools] rtemstoolkit: add support for executing pipe command

2023-06-19 Thread Chris Johns
Hi, Thanks for this. I like the change. Could you please shorten the long line? This code will be run through yapf after 6 branches but until then we limit the line length. :) Could you please add a unit test for this to the end of this file? Thanks Chris On 20/6/2023 8:44 am, Muhammad

[PATCH rtems-tools] rtemstoolkit: add support for executing pipe command

2023-06-19 Thread Muhammad Sulthan Mazaya
The previous implementation of execute command on rtems-tools config does not support command line pipe operation. So something like: %execute %{command} | %{trim_command} Would not work, since the "| %{trim_command}" part is treated as an additional command option. This patch is intended to