3 ms·
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 p
by JdeBP 1y ago
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/patches/0012-Don-t-try-to-exec-char-1-if-SHELL-doesn-t-have-a-Clo.patch/ https://sources.debian.org/src/ttyrec/1.1.7.1-1/debian/patch...
* https://github.com/debba/rewindtty/blob/develop/src/recorder.c#L167 https://github.com/debba/rewindtty/blob/develop/src/recorder...
* https://github.com/mjording/ttyrec/blob/master/ttyrec.c#L328 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 1y agoBy the way, debba: execvp()/execlp() and _PATH_BSHELL out of <paths.h> as the fallback are the ways to go, here.
- debba 1y agoOuch 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.