← Back to context

Comment by loloquwowndueo

1 day ago

Is this ttyrec with json output?

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. (-:

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

    • 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.

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