pypa/pipx

cli parameter parsing is corrupted by strict positional

Chiusa

#1444 aperta il 7 giu 2024

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Python (591 fork)batch import
bughelp wanted

Metriche repository

Star
 (12.930 stelle)
Metriche merge PR
 (Merge medio 4g 12h) (22 PR mergiate in 30 g)

Descrizione

In version 1.6.0, the following command cannot be used

Describe the bug

In version 1.6.0, the following command cannot be used.

pipx inject ansible-core --force ansible

error: unrecognized arguments: ansible

While adjusting the order can work

pipx inject ansible-core ansible --force

But this will create compatibility issues, the current way used in devcontainers is

features/src/pipx-package/install.sh at 9a1d24b27b2d1ea8916ebe49c9ce674375dced27 · devcontainers-contrib/features

[Bug]: Failed to build ansible feature · Issue #607 · devcontainers-contrib/features (github.com)

This is related to HIGH: pipx install --global order broken in 1.6.0 · Issue #1443 · pypa/pipx

Guida contributor

Direzione di ricerca
Investigate the CLI argument parsing code, likely in the `pipx/` directory. Compare the behavior of `pipx inject` with and without ` force` positional argument. The issue is related to a change in version 1.6.0 that introduced strict positional parsing. Look at the argparse configuration for the inject subcommand. Also review the linked issue #1443 for context on the global command being broken.
Tech stack
python
Dominio
cli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Pythonpipx usageCLI argument parsing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85