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

cut: Fix cut when lines dont end with specified delim #5844

Merged
merged 3 commits into from
Dec 3, 2024

Conversation

andreistan26
Copy link
Contributor

issue: #5795

Using the default line terminator \n and without the additional option --only-delimited, cut should print the lines that do not contain delimiters only when they are ended with the specified line terminator(the default \n in this case or null-terminated if --zero-terminated is enabled).

In the given input the last line was not ended in a \n so it shouldn't have been printed.

Print lines without delimiters only when they end with specified line terminator('\n' by default or `\0` if `-s`)

Signed-off-by: Andrei Stan <andreistan2003@gmail.com>
@cakebaker cakebaker changed the title Print lines without delimiters only when they end with specified line… cut: Print lines without delimiters only when they end with specified line… Jan 15, 2024
@andreistan26 andreistan26 changed the title cut: Print lines without delimiters only when they end with specified line… cut: Fix cut when lines dont end with specified delim Jan 15, 2024
@cakebaker cakebaker linked an issue Jan 15, 2024 that may be closed by this pull request
Copy link

GNU testsuite comparison:

GNU test failed: tests/timeout/timeout. tests/timeout/timeout is passing on 'main'. Maybe you have to rebase?
Congrats! The gnu test tests/mv/mv-n is no longer failing!
Congrats! The gnu test tests/mv/update is no longer failing!

Copy link

github-actions bot commented Dec 2, 2024

GNU testsuite comparison:

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

@sylvestre sylvestre merged commit 5b087e9 into uutils:main Dec 3, 2024
62 checks passed
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.

cut: different output with -d
2 participants