Re: [libvirt PATCH 09/12] ci: jobs.sh: run_cmd: Use eval to run commands

2023-09-18 Thread Daniel P . Berrangé
On Mon, Sep 18, 2023 at 01:47:03PM +0200, Erik Skultety wrote: > On Mon, Sep 18, 2023 at 11:31:53AM +0100, Daniel P. Berrangé wrote: > > On Mon, Sep 18, 2023 at 12:22:45PM +0200, Erik Skultety wrote: > > > We tried to evade usage of eval in commit 6214ae55f6a, but trying to > > > use I/O

Re: [libvirt PATCH 09/12] ci: jobs.sh: run_cmd: Use eval to run commands

2023-09-18 Thread Erik Skultety
On Mon, Sep 18, 2023 at 11:31:53AM +0100, Daniel P. Berrangé wrote: > On Mon, Sep 18, 2023 at 12:22:45PM +0200, Erik Skultety wrote: > > We tried to evade usage of eval in commit 6214ae55f6a, but trying to > > use I/O redirections with a command doesn't have the desired effect, > > because when

Re: [libvirt PATCH 09/12] ci: jobs.sh: run_cmd: Use eval to run commands

2023-09-18 Thread Daniel P . Berrangé
On Mon, Sep 18, 2023 at 12:22:45PM +0200, Erik Skultety wrote: > We tried to evade usage of eval in commit 6214ae55f6a, but trying to > use I/O redirections with a command doesn't have the desired effect, > because when Shell eats the redirection it is applied to anything > inside the run_cmd

[libvirt PATCH 09/12] ci: jobs.sh: run_cmd: Use eval to run commands

2023-09-18 Thread Erik Skultety
We tried to evade usage of eval in commit 6214ae55f6a, but trying to use I/O redirections with a command doesn't have the desired effect, because when Shell eats the redirection it is applied to anything inside the run_cmd function, even the print command we use for debugging purposes. In order to