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 command printing extra line that was not ending with the specified terminator #6

Open
wants to merge 1,564 commits into
base: main
Choose a base branch
from

Conversation

andreistan26
Copy link

@andreistan26 andreistan26 commented Jan 8, 2024

issue: uutils#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.

@alexandruradovici
Copy link

Please rebase this and send it to upstream, and place a link to it here.

@andreistan26 andreistan26 force-pushed the cut-newline branch 2 times, most recently from 09848bf to 544ac28 Compare January 15, 2024 14:26
@andreistan26
Copy link
Author

uutils/pull/5844

sylvestre and others added 26 commits October 1, 2024 14:05
chore(deps): update rust crate lscolors to 0.20.0
* fix issue uutils#6248

* add test to cat for the case of issue uutils#6248
ci: use ubuntu v24 for GNU tests workflow
cat: use `write_end_of_line()`  to avoid duplicate code
…d_of_vec

users: use `Option` instead of `Vec` for file arg
…ding

basenc: perform faster, streaming encoding
…kfile

chore(deps): update rust crate once_cell to v1.20.2
* fix uutils#5186 by adding explicit flush.
* make test machine-independent

---------

Co-authored-by: Sylvestre Ledru <[email protected]>
Co-authored-by: Daniel Hofstetter <[email protected]>
* join: add test for multibyte separators

* join: implement support for multibyte separators

* join: use a trait instead of an enum for separator

* join: test whitespace merging
…-vm-1.x

chore(deps): update vmactions/freebsd-vm action to v1.1.4
chore(deps): update rust crate procfs to 0.17
users: fix lint error manual_unwrap_or_default on OpenBSD
…ockfile

fix(deps): update rust crate proc-macro2 to v1.0.87
add back use_self clippy warning
cakebaker and others added 30 commits November 27, 2024 07:02
fix(deps): update rust crate libc to v0.2.166
chore(deps): update rust crate blake3 to v1.5.5
…to_tests

env: add missing `cfg` attributes to tests
* tr: Add ambiguous octal escape warning, issue uutils#6821

* tr: Make code cleaner
The following explicit lifetimes could be elided: 'a
Should fix:
tests/du/hard-link.sh
…wnload-artifact-7.x

chore(deps): update dawidd6/action-download-artifact action to v7
fix(deps): update rust crate libc to v0.2.167
…-lockfile

chore(deps): update rust crate terminal_size to v0.4.1
…etimes

uudoc,chcon: fix `needless_lifetimes` warnings
* tr: generate an error if the input is a directory

tested by tests/misc/read-errors

* tr: improve the test

* tr: take the commentinto account
comm: generate an error if the input is a directory
uucore/perms: use ORs instead of `match` (fix todo)
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.