8000 multicall: add --list to list all utils by tertsdiepraam · Pull Request #6251 · uutils/coreutils · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

multicall: add --list to list all utils #6251

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 1, 2024
Merged

multicall: add --list to list all utils #6251

merged 1 commit into from
May 1, 2024

Conversation

tertsdiepraam
Copy link
Member

Closes #6250

I've implemented it more or less like the busybox version now, by just matching on the string --list. Ultimately, we might want to go for more sophisticated argument parsing in the multicall binary.

It is a bit strange though, because all the other things that the multicall binary can do are implemented as subcommands. In other words, one of these is not like the others:

coreutils completion
coreutils manpage
coreutils --list

I think using -- makes sense to clearly separate it from the utils. Maybe completion and manpage should be prefixed with -- too?

@sylvestre
Copy link
Contributor

is that testable ? :)
without a test, it feels fragile

@tertsdiepraam
Copy link
Member Author

Maybe, I'll look into that later today.

@BenWiederhake
Copy link
Collaborator

Related to testability: #6198

@sylvestre sylvestre merged commit 45bf199 into uutils:main May 1, 2024
67 of 68 checks passed
@tertsdiepraam tertsdiepraam mentioned this pull request Jun 19, 2024
Closed
@BrewTestBot BrewTestBot mentioned this pull request Jun 23, 2024
Merged
@moonfruit moonfruit mentioned this pull request Jun 24, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

--list for the multicall binary
3 participants
@tertsdiepraam @sylvestre @BenWiederhake
0