Page MenuHomeDevCentral

Offer an option to append or prepend the line
ClosedPublic

Authored by dereckson on Oct 2 2018, 21:30.
Tags
None
Referenced Files
F3936115: D1837.diff
Mon, Dec 23, 10:26
Unknown Object (File)
Sun, Dec 22, 10:23
Unknown Object (File)
Sat, Dec 14, 11:27
Unknown Object (File)
Sat, Dec 14, 11:09
Unknown Object (File)
Thu, Dec 5, 22:20
Unknown Object (File)
Wed, Dec 4, 17:45
Unknown Object (File)
Mon, Dec 2, 07:20
Unknown Object (File)
Sat, Nov 30, 07:46
Subscribers
None

Details

Summary

The output is clearly more legible when the container name is put
before the text, especially for ps where arguments could be long.

By default, the command prepends 20 characters with container name
and spaces. This behavior can be modified by the --append flag
and the -w <number of characters> argument.

Ref T1456

Test Plan
  • test prepend/append modes
  • check how container name is truncated
  • non UTF-8 console (… is replaced by +)

Diff Detail

Repository
rDP Docker Processes
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
prepend (branched from master)
Build Status
Buildable 2834
Build 3082: arc lint + arc unit

Event Timeline

dereckson created this revision.
dereckson marked an inline comment as done.
dereckson added inline comments.
internal/consoleutilities/consoleutilities.go
9

man 7 environ indicated LC_ALL is an override for those two.

This revision is now accepted and ready to land.Oct 3 2018, 09:49
This revision was automatically updated to reflect the committed changes.