Skip to content
  • Jeff King's avatar
    stash: complain about unknown flags · d6cc2df5
    Jeff King authored and Junio C Hamano's avatar Junio C Hamano committed
    
    
    The option parser for git-stash stuffs unknown flags into
    the $FLAGS variable, where they can be accessed by the
    individual commands. However, most commands do not even look
    at these extra flags, leading to unexpected results like
    this:
    
      $ git stash drop --help
      Dropped refs/stash@{0} (e6cf6d80faf92bb7828f7b60c47fc61c03bd30a1)
    
    We should notice the extra flags and bail. Rather than
    annotate each command to reject a non-empty $FLAGS variable,
    we can notice that "stash show" is the only command that
    actually _wants_ arbitrary flags. So we switch the default
    mode to reject unknown flags, and let stash_show() opt into
    the feature.
    
    Reported-by: default avatarVincent Legoll <vincent.legoll@gmail.com>
    Signed-off-by: default avatarJeff King <peff@peff.net>
    Signed-off-by: default avatarJunio C Hamano <gitster@pobox.com>
    d6cc2df5