Skip to content

Deprecated stcatv and add stcatn#23

Merged
adrelanos merged 1 commit intoKicksecure:masterfrom
ben-grande:stcatn
Apr 16, 2025
Merged

Deprecated stcatv and add stcatn#23
adrelanos merged 1 commit intoKicksecure:masterfrom
ben-grande:stcatn

Conversation

@ben-grande
Copy link
Copy Markdown
Contributor

Deprecate stcatv:

  • The 'M-' and '^' notation were not covering all unicode codepoints, issue is present in cat from coreutils also.

Add stcatn:

  • Printing data without ending with a newline can lead to problems on the terminal and also how the file is read by another problem later. Add a newline at the end of each file that does not have one.
  • Trailing whitespace is trimmed from each line.

This pull request changes...

Changes

Mandatory Checklist

  • Legal agreements accepted. By contributing to this organisation, you acknowledge you have read, understood, and agree to be bound by these these agreements:

Terms of Service, Privacy Policy, Cookie Policy, E-Sign Consent, DMCA, Imprint

Optional Checklist

The following items are optional but might be requested in certain cases.

  • I have tested it locally
  • I have reviewed and updated any documentation if relevant
  • I am providing new code and test(s) for it

Fixes #

Fixes: #22
Fixes: #21
Fixes: #20

Deprecate stcatv:

- The 'M-' and '^' notation were not covering all unicode codepoints,
  issue is present in cat from coreutils also.

Add stcatn:

- Printing data without ending with a newline can lead to problems on
  the terminal and also how the file is read by another problem later.
  Add a newline at the end of each file that does not have one.
- Trailing whitespace is trimmed from each line.

Fixes: Kicksecure#22
Fixes: Kicksecure#21
Fixes: Kicksecure#20
@adrelanos adrelanos merged commit 08eb6b2 into Kicksecure:master Apr 16, 2025
5 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

2 participants