Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cli parameter parsing is corrupted by strict positional #1444

Open
xiaoyao9184 opened this issue Jun 7, 2024 · 2 comments
Open

cli parameter parsing is corrupted by strict positional #1444

xiaoyao9184 opened this issue Jun 7, 2024 · 2 comments
Labels
help wanted Extra attention is needed

Comments

@xiaoyao9184
Copy link

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

@dukecat0
Copy link
Member

dukecat0 commented Jun 7, 2024

@huxuan
Copy link
Member

huxuan commented Jun 8, 2024

Probably it's because of the change of this line: https://github.com/pypa/pipx/pull/1252/files#diff-be12955f38ede5e93b9a5868a5c13533b561c74acc9686d2e886cc9428148e32R519

Seems there is no easy solution for back-compatibility while keeping the --requirement feature. :-(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants