Hi Robert,

if you feel that's a bug, would you open a pull request upstream?

Kind regards,
 Valentin

On Mon, Jan 27, 2020 at 10:30 PM Robert P. J. Day <rpjday@crashcourse.ca> wrote:

  just noticed (and verified) the following ... while "man
podman-stop" documents the three variations for the timeout option:

   --timeout, --time, t=time

   Timeout to wait before forcibly stopping the container

the code itself associates "-t" with "--timeout", as opposed to
"--time", and checks for conflicts with:

        flags.UintVar(&stopCommand.Timeout, "time",
define.CtrRemoveTimeout, "Seconds to wait for stop before killing the
container")
        flags.UintVarP(&stopCommand.Timeout, "timeout", "t",
define.CtrRemoveTimeout, "Seconds to wait for stop before killing the
container")

... snip ...

func stopCmd(c *cliconfig.StopValues) error {
        if c.Flag("timeout").Changed && c.Flag("time").Changed {
                return errors.New("the --timeout and --time flags are mutually exclusive")
        }

  amusingly, the above appears to allow the combination of "--timeout"
and "-t" with no problem, which seems counter-intuitive.

rday

--

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                         http://crashcourse.ca

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================
_______________________________________________
Podman mailing list -- podman@lists.podman.io
To unsubscribe send an email to podman-leave@lists.podman.io