Return an error if the same flag is provided twice with different values #72

Activity

1. n10v

   added a commit that references this issue on Mar 26, 2017

   Show error if flag is provided multiple times

   ...

   This issue will close once commit 53c44ff is merged into the 'master' branch.

   53c44ff
2. eparis commented on Mar 26, 2017

   

   eparis

   on Mar 26, 2017

   Collaborator

   More actions

   I feel like that is a custom flag type. I don't know of any unix command that complains when the same flag is given twice. All of them I know of either use both values or the last value, depending on the type of flag...

   👍React with 👍4Reacted by Albert, Kai Davenport, John Schnake and Adam S Levy
3. ncw commented on Mar 27, 2017

   

   ncw

   on Mar 27, 2017

   Author

   More actions

   Now that pflag supports StringArray and friends I'm now using those in rclone so this issue is less important for me personally now.

   However duplicated flags especially with arguments, when the parser can't cope with them, should make an error. Just because getopt(3) is lazy doesn't mean we have to be!
4. dixudx commented on Nov 13, 2017

   

   dixudx

   on Nov 13, 2017

   More actions

   I think we should apply this as default for all non-sliced types.

   PTAL #151. I introduced a new method AllowingMultipleSet() to Flag. For all sliced/array types, this function is enforced as default.
5. normanr commented on May 10, 2020

   

   normanr

   on May 10, 2020

   Last edited by normanr

   More actions

   I feel like that is a custom flag type. I don't know of any unix command that complains when the same flag is given twice. All of them I know of either use both values or the last value, depending on the type of flag...

   here's an example (not a core unix command, but it's on most systems these days)

   $ ssh localhost -W host:1234 -W host:1235
   stdio forward already specified
   
6. dmilinevskyi commented on May 31, 2024

   

   dmilinevskyi

   on May 31, 2024

   More actions

   I upvote this feature request.

   Here's my use case: depending on the context of a sub-command, some flags might have different meaning.
   In one case --foo can be given only once, while in other cases --foo might be given multiple times.
   It might be confusing for a user if it's allowed to specify multiple --foo but only the last value is used.
   I agree that it's reasonable to argue that it might be a bad design, but in some cases reworking an application to add a new flag --foos or converting all --foo to array and checking the length manually and failing if len(foo) != 1 might not be acceptable either.

   The other use case is that the users might copy/paste or reuse commands from history and modify some of the arguments. It might happen that the command looks like --foo --bar ..... --zoo=A and the user wants to replace --bar with --bin --zoo=B. The user might be surprised by the result.

   I'm happy to help designing this feature and working on PRs.
   It seems like it's a reasonably small change, it's possible to reuse existing Changed field

   pflag/flag.go

   Line 481 in d5e0c06

   if !flag.Changed {