-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
fmt: clap causes information loss #6353
Labels
Comments
Yep this is one of the cases that almost takes another parser. |
Something very similar happens with
Empty list ignores nothing: $ env --ignore-signal= sleep 999 &
[1] 564133
$ ps | grep sleep
564133 pts/9 00:00:00 sleep
$ kill -s SIGHUP 564133
$
[1]+ Hangup env --ignore-signal= sleep 999 Providing no argument ignores everything: $ env --ignore-signal sleep 999 &
[1] 564145
$ ps | grep sleep
564145 pts/9 00:00:00 sleep
$ kill -s SIGHUP 564145
$
$ sleep 1
$ ps | grep sleep # Still alive!
564145 pts/9 00:00:00 sleep Found while reviewing #6377. |
BenWiederhake
added a commit
to BenWiederhake/uutils-args
that referenced
this issue
May 9, 2024
Note in particular that this easily solves the issue of clap's information loss: uutils/coreutils#6353
BenWiederhake
added a commit
to BenWiederhake/uutils-args
that referenced
this issue
May 9, 2024
Note in particular that this easily solves the issue of clap's information loss: uutils/coreutils#6353
BenWiederhake
added a commit
to BenWiederhake/uutils-args
that referenced
this issue
May 9, 2024
Note in particular that this easily solves the issue of clap's information loss: uutils/coreutils#6353
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Or in general: GNU parses
-p=ANYTHING
to be the prefix=ANYTHING
, whereas clap parses the same argument as the different stringANYTHING
. This cannot be easily recovered, and basically requires parsing the arguments from scratch again.This also affects many other tools. I think it was already documented somewhere, but I couldn't find it.
I see this as yet another strong argument in favor of #4254.
The text was updated successfully, but these errors were encountered: