search

npat-efault / picocom

Minimal dumb-terminal emulation program
GNU General Public License v2.0
644 stars 126 forks source link

Add ability to print lines with timestamp #125

Open ismell opened 3 years ago

ismell commented 3 years ago

There is a PR here that adds such functionality: https://github.com/codepox/picocom/pull/1

scruss commented 3 years ago

this is something that grabserial — https://github.com/tbird20d/grabserial — already does well, so maybe picocom doesn't need it.

ismell commented 3 years ago

Ah, that looks promising. I'm fine if we close this issue.

On Sat, Apr 24, 2021, at 8:41 AM, Stewart C. Russell wrote:

this is something that grabserial — https://github.com/tbird20d/grabserial — already does well, so maybe picocom doesn't need it.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/npat-efault/picocom/issues/125#issuecomment-826102831, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEJOIXAASZKYSVTB6FFETTTKLKCHANCNFSM42JNRYCQ.

vbrozik commented 2 years ago

this is something that grabserial — https://github.com/tbird20d/grabserial — already does well, so maybe picocom doesn't need it.

As I understand it from the description grabserial is for something else - it records non-interactive serial communication.

wsakernel commented 2 years ago

After applying https://github.com/wsakernel/picocom/commit/bd4de4857dedd23e0ae520068f3143b2df196c6f, ninja-ts works great as a pipe with picocom for me. Without the patch, it still works, just has some garbled output at the beginning. I don't think we need extra code for that in picocom.