From: bors <bors@rust-lang.org>
Date: Fri, 16 Mar 2018 15:08:24 +0000 (+0000)
Subject: Auto merge of #5196 - matklad:clapclapclap, r=alexcrichton
X-Git-Tag: archive/raspbian/0.35.0-2+rpi1~3^2^2^2^2^2^2^2~22^2~2^2~34
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=d6c3983fe3bd8fa06b54712e53fb23645598188b;p=cargo.git

Auto merge of #5196 - matklad:clapclapclap, r=alexcrichton

Fix a regression with parsing multivalue options

By default, clap interprets

```
cargo run --bin foo bar baz
```

as

```
cargo run --bin foo --bin bar --bin baz
```

This behavior is different from docopt and does not play nicely with
positional arguments at all. Luckily, clap has a flag to get the
behavior we want, it just not the default! It will become the default in
the next version of clap, but, until that time, we should be careful
when using the combination of `.long`, `.value_name` and
`.multiple(true)`, and don't forget to specify `.number_of_values(1)` as
well.

@alexcrichton I'd love to merge this fix before updating cargo at rust-lang/rust :)
---

d6c3983fe3bd8fa06b54712e53fb23645598188b