dep ensure does not accept -v with -update

[davecheney]

What version of dep are you using (dep version)?

v0.3.2-83-g13df5561

What dep command did you run?

dep ensure -update k8s.io/[email protected] -v

What did you expect to see?

verbose output of the update process

What did you see instead?

Invalid arguments passed to ensure -update:

  ✗ could not infer project root from dependency path: -v: "-v" is not a valid import path
  ✗ version constraint ^6.0.0 passed for k8s.io/client-go, but -update follows constraints declared in Gopkg.toml, not CLI arguments

update arguments validation failed

[avgeorge]

Setting -v first works as expected for me.
Try dep ensure -v -update k8s.io/[email protected].
Trying to set -v after -update gives me the first error you see ("could not infer project root ...")

[carolynvs]

Thanks for reporting the bug in our flag parsing!

[awoodbeck]

I can take this one. The stdlib flag package stops parsing command line options once it encounters an argument that isn't a flag. Short of using an alternative flag package or augmenting the way dep uses the flag package, the correct fix for this issue may simply be updating cmd.Args() to ensure the optional -v flag precedes the optional [<spec>...] argument.

[davecheney]

This could also be fixed by documenting -v in the sub command syntax. It is currently not documented, only inferred from other commands.

…

On 27 Dec 2017, at 11:49, Adam Woodbeck ***@***.***> wrote: I can take this one. The stdlib flag package stops parsing command line options once it encounters an argument that isn't a flag. Short of using an alternative flag package or augmenting the way dep uses the flag package, the correct fix for this issue may simply be updating cmd.Args() to ensure the optional -v flag precedes the optional [<spec>...] argument. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

[awoodbeck]

@davecheney I think we were talking about the same thing but my terminology was different. The -v option is in the sub command's flag set and shows up in the list of options. But its use isn't detailed in the sub command's syntax. The PR I just proposed addresses this.