Unrecognized argument when multiple different actions have the same flag value #881

New Issue

Closed

opened 2024-06-20 18:51:34 +00:00 by witten · 2 comments

witten commented 2024-06-20 18:51:34 +00:00

Owner

Copy Link

What I'm trying to do and why

I'm trying to run a borgmatic action that receives a command-line flag (as part of testing an unrelated ticket's implementation), but I'm receiving an error: Unrecognized argument: 1 where 1 is the flag's value.

Steps to reproduce

borgmatic -c test.yaml -v 1 check --match-archives 1

Actual behavior

usage: borgmatic [-h] [-c CONFIG_PATHS] [-n] [-nc] [-v {-2,-1,0,1,2}] [--syslog-verbosity {-2,-1,0,1,2}] [--log-file-verbosity {-2,-1,0,1,2}]
                 [--monitoring-verbosity {-2,-1,0,1,2}] [--log-file LOG_FILE] [--log-file-format LOG_FILE_FORMAT] [--log-json]
                 [--override OPTION.SUBOPTION=VALUE] [--no-environment-interpolation] [--bash-completion] [--fish-completion] [--version]
                 ...
Unrecognized argument: 1

Need some help? https://torsion.org/borgmatic/#issues

Expected behavior

The arguments parse and the borgmatic check action proceeds.

Other notes / implementation ideas

This only happens when a flag's value is present for multiple actions in the command-line. For instance, here -v 1 is a flag and value for the global action and --match-archives 1 is a flag and (the same) value for the check action. But if the --match-archives value is changed to anything else, there is no error.

This occurs because of an unfortunate bug in get_unparsable_arguments(); it's blindly looking for any argument strings that show up across actions, treating them as "unparsable" arguments that no actions have consumed. And those get reported as unrecognized arguments to the user.

borgmatic version

1.8.11 or main

borgmatic installation method

pipx install --editable

Borg version

1.2.8

Python version

3.12.3

Database version (if applicable)

No response

Operating system and version

Manjaro stable

### What I'm trying to do and why I'm trying to run a borgmatic action that receives a command-line flag (as part of testing an unrelated ticket's implementation), but I'm receiving an error: `Unrecognized argument: 1` where `1` is the flag's value. ### Steps to reproduce ```bash borgmatic -c test.yaml -v 1 check --match-archives 1 ``` ### Actual behavior ``` usage: borgmatic [-h] [-c CONFIG_PATHS] [-n] [-nc] [-v {-2,-1,0,1,2}] [--syslog-verbosity {-2,-1,0,1,2}] [--log-file-verbosity {-2,-1,0,1,2}] [--monitoring-verbosity {-2,-1,0,1,2}] [--log-file LOG_FILE] [--log-file-format LOG_FILE_FORMAT] [--log-json] [--override OPTION.SUBOPTION=VALUE] [--no-environment-interpolation] [--bash-completion] [--fish-completion] [--version] ... Unrecognized argument: 1 Need some help? https://torsion.org/borgmatic/#issues ``` ### Expected behavior The arguments parse and the borgmatic `check` action proceeds. ### Other notes / implementation ideas This only happens when a flag's value is present for multiple actions in the command-line. For instance, here `-v 1` is a flag and value for the global action and `--match-archives 1` is a flag and (the same) value for the `check` action. But if the `--match-archives` value is changed to anything else, there is no error. This occurs because of an unfortunate bug in `get_unparsable_arguments()`; it's blindly looking for any argument strings that show up _across_ actions, treating them as "unparsable" arguments that no actions have consumed. And those get reported as unrecognized arguments to the user. ### borgmatic version 1.8.11 or main ### borgmatic installation method pipx install --editable ### Borg version 1.2.8 ### Python version 3.12.3 ### Database version (if applicable) _No response_ ### Operating system and version Manjaro stable

witten added the bug label 2024-06-20 18:51:38 +00:00

witten referenced this issue from a commit 2024-06-20 20:32:08 +00:00

Fix "Unrecognized argument" error when the same value is with different command-line flags (#881).

witten commented 2024-06-20 20:32:45 +00:00

Author

Owner

Copy Link

Fixed in main. Will be part of the next release.

Fixed in main. Will be part of the next release.

witten closed this issue 2024-06-20 20:32:53 +00:00

witten referenced this issue from a commit 2024-06-20 20:33:57 +00:00

Added missing word to NEWS entry (#881).

witten commented 2024-06-24 00:06:23 +00:00

Author

Owner

Copy Link

Released in borgmatic 1.8.12!

Released in borgmatic 1.8.12!

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

main

command-hook-steps

2.0.11

2.0.10

2.0.9

2.0.8

2.0.7

2.0.6

2.0.5

2.0.4

2.0.3

2.0.2

2.0.1

2.0.0

1.9.14

1.9.13

1.9.12

1.9.11

1.9.10

1.9.9

1.9.8

1.9.7

1.9.6

1.9.5

1.9.4

1.9.3

1.9.2

1.9.1

1.9.0

1.8.14

1.8.13

1.8.12

1.8.11

1.8.10

1.8.9

1.8.8

1.8.7

1.8.6

1.8.5

1.8.4

1.8.3

1.8.2

1.8.1

1.8.0

1.7.15

1.7.14

1.7.13

1.7.12

1.7.11

1.7.10

1.7.9

1.7.8

1.7.7

1.7.6

1.7.5

1.7.4

1.7.3

1.7.2

1.7.1

1.7.0

1.6.6

1.6.5

1.6.4

1.6.3

1.6.2

1.6.1

1.6.0

1.5.24

1.5.23

1.5.22

1.5.21

1.5.20

1.5.19

1.5.18

1.5.17

1.5.16

1.5.15

1.5.14

1.5.13

1.5.12

1.5.11

1.5.10

1.5.9

1.5.8

1.5.7

1.5.7.dev0

1.5.6

1.5.5

1.5.4

1.5.3

1.5.2

1.5.1

1.5.0

1.4.22

1.4.21

1.4.20

1.4.19

1.4.18

1.4.17

1.4.16

1.4.15

1.4.14

1.4.13

1.4.12

1.4.11

1.4.10

1.4.9

1.4.8

1.4.7

1.4.6

1.4.5

1.4.4

1.4.3

1.4.2

1.4.1

1.4.0

1.3.26

1.3.25

1.3.24

1.3.23

1.3.22

1.3.21

1.3.20

1.3.19

1.3.18

1.3.17

1.3.16

1.3.15

1.3.14

1.3.13

1.3.12

1.3.11

1.3.10

1.3.9

1.3.8

1.3.7

1.3.6

1.3.5

1.3.4

1.3.3

1.3.2

1.3.1

1.3.0

1.2.18

1.2.17

1.2.16

1.2.15

1.2.14

1.2.13

1.2.12

1.2.11

1.2.10

1.2.9

1.2.8

1.2.7

1.2.6

1.2.5

1.2.4

1.2.3

1.2.2

1.2.1

1.2.0

1.1.15

1.1.14

1.1.13

1.1.12

1.1.11

1.1.10

1.1.9

1.1.7

1.1.6

1.1.5

1.1.4

1.1.3

1.1.2

1.1.1

1.1.0

1.0.3

1.0.2

1.0.1

1.0.0

0.1.8

0.1.7

0.1.6

0.1.5

0.1.4

0.1.3

0.1.2

0.1.1

0.1.0

0.0.7

0.0.6

0.0.5

0.0.4

0.0.3

0.0.2

Labels

Clear labels

blocked

breaking

bug

data loss

design finalized

good first issue

new feature area

question / support

security

waiting for response

No Label bug

Milestone

No items

No Milestone

Assignees

Clear assignees

witten

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#881