1808584 - patch command has `--diff-id` and `--apply-to` as mutually exclusive

Status: RESOLVED FIXED

(Conduit :: moz-phab, defect)

Description

[Connor Sheehan [:sheehan]]

❯ moz-phab patch D165893 --apply-to a0ec48ed7110fd84c84d1b5cca0d87bed2115d7f --diff-id 664407
usage: moz-phab patch [-h] [--apply-to TARGET | --diff-id DIFF_ID | --raw]
                      [--name NAME] [--no-commit] [--no-bookmark] [--no-topic]
                      [--no-branch] [--skip-dependencies]
                      [--include-abandoned] [--yes] [--safe-mode]
                      [--force-vcs]
                      revision_id
moz-phab patch: error: argument --diff-id: not allowed with argument --apply-to/--applyto/-a

This is because we are using the mutually exclusive patch parser group instead of the standard parser. Patch incoming.

Comment 1

[Connor Sheehan [:sheehan]]

Attachment: patch: move `--diff-id` argument out of mutually exclusive group (Bug 1808584) r?grammar

Move the --diff-id argument out of the mutually exclusive
group meant for --apply-to and --raw. Also add a comment
about the mutually exclusive group and add spaces around them
so the different parser is more obvious.