Remix.run Logo
loloquwowndueo 3 days ago

Is this ttyrec with json output?

JdeBP 3 days ago | parent | next [-]

No.

Because checking some of the Debian patches accrued against ttyrec, and the source to rewindtty, it seems that rewindtty does not have some of the common programming mistakes that the author of ttyrec made a quarter of a century ago.

* https://sources.debian.org/src/ttyrec/1.1.7.1-1/debian/patch...

* https://github.com/debba/rewindtty/blob/develop/src/recorder...

* https://github.com/mjording/ttyrec/blob/master/ttyrec.c#L328

rewindtty will error out if the SHELL environment variable's value does not have a directory prefix, but at least it won't outright crash. (-:

JdeBP 3 days ago | parent [-]

By the way, debba: execvp()/execlp() and _PATH_BSHELL out of <paths.h> as the fallback are the ways to go, here.

debba 3 days ago | parent [-]

Ouch sorry, I thought you meant 'it has the same features as ttyrec with JSON output.' Regarding the issue, I need to go into a bit more detail; if you like, feel free to make a PR on GitHub.

debba 3 days ago | parent | prev | next [-]

sorry, in the last reply I told you 'yes, exactly'. I meant It has the same features as ttyrec with JSON output

debba 3 days ago | parent | prev [-]

yes exactly