Fix stop/restart args stacking

[dkull]

For reboots, arguments like '-t X' are stacked as many times as there are services, because the args list is created before the for loop iterating over the containers, and the '-t X' is extended into the arguments list.

So if my Composefile.yml has services: alpha, bravo, charlie
and I do podman-compose -f Composefile.yml restart -t 1
it would output roughly:

podman restart -t 1 alpha
podman restart -t 1 -t 1 bravo
podman restart -t 1 -t 1 -t 1 charlie

The fix moves the creation of the list, that collects the args, into the loop itself. This enables it to extend the list for each loop iteration.

[maurerle]

I would love to see this merged.

[p12tic]

Looks great!

[p12tic]

Only my gravatar is visible on the commit after rebase, but this is because there's no author gravatar. I didn't change authorship.

[p12tic]

Normally I would ask for tests, but this is so simple change which is immediately useful to users while the change is in an area that is relatively hard to unit-test. So in this particular case I merge the PR as is.