Skip to content

cut: fix handling of newline as delimiter #6996

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
sylvestre merged 1 commit into from
Dec 27, 2024
Merged

cut: fix handling of newline as delimiter #6996

sylvestre merged 1 commit into from
Dec 27, 2024

Conversation

@cakebaker
Copy link
Contributor

@cakebaker cakebaker commented Dec 24, 2024

This PR fixes the handling of newline as a delimiter and should make all tests in cut.pl pass.

@github-actions
Copy link

github-actions bot commented Dec 24, 2024

GNU testsuite comparison:

Congrats! The gnu test tests/cut/cut is no longer failing!
Skipping an intermittent issue tests/tail/inotify-dir-recreate (passes in this run but fails in the 'main' branch)

@sylvestre
Copy link
Contributor

sylvestre commented Dec 26, 2024

Congrats! The gnu test tests/cut/cut is no longer failing!

wahou, kudos :)

sylvestre
sylvestre reviewed Dec 26, 2024
View reviewed changes
@cakebaker cakebaker force-pushed the cut_newline_as_delimiter branch from 7bacd84 to 20dfb27 Compare December 27, 2024 08:12
@github-actions
Copy link

github-actions bot commented Dec 27, 2024

GNU testsuite comparison:

Congrats! The gnu test tests/cut/cut is no longer failing!
Skipping an intermittent issue tests/tail/inotify-dir-recreate (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/timeout/timeout (passes in this run but fails in the 'main' branch)

@sylvestre sylvestre merged commit 4809ece into uutils:main Dec 27, 2024
61 of 62 checks passed
@cakebaker cakebaker deleted the cut_newline_as_delimiter branch December 27, 2024 08:45
@moonfruit moonfruit mentioned this pull request Jan 20, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sylvestre sylvestre sylvestre left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cakebaker @sylvestre