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

timeout: add support for -f and -p short options #7200

Merged
merged 2 commits into from
Jan 23, 2025

Conversation

cakebaker
Copy link
Contributor

@cakebaker cakebaker commented Jan 23, 2025

This PR adds support for -f and -p. These short options have been added in GNU timeout 9.6. From the release notes:

timeout now supports the POSIX:2024 specified -f, and -p short options,
corresponding to --foreground, and --preserve-status respectively.

The PR also simplifies some tests by using no_output() instead of no_stderr() and no_stdout().

Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/timeout/timeout (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/stdbuf (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/misc/usage_vs_getopt (passes in this run but fails in the 'main' branch)

@sylvestre sylvestre merged commit 704421b into uutils:main Jan 23, 2025
65 checks passed
@cakebaker cakebaker deleted the timeout_add_f_and_p branch January 24, 2025 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants