[libc-commits] [PATCH] D136143: [libc] Add termios.h and the implementation of functions declared in it.
Siva Chandra via Phabricator via libc-commits
libc-commits at lists.llvm.org
Tue Oct 18 11:44:32 PDT 2022
sivachandra marked an inline comment as done.
sivachandra added inline comments.
================
Comment at: libc/test/src/termios/termios_test.cpp:48
+ errno = 0;
+ int fd = __llvm_libc::open("/dev/tty", O_RDONLY);
+ ASSERT_EQ(errno, 0);
----------------
michaelrj wrote:
> this will fail on the buildbot, you need to put the rest of the test inside an `if(fd > 0)` clause, similar to how `isatty_test.cpp` does it.
Can we try once more on the bot? If it fails, I will add a conditional here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136143/new/
https://reviews.llvm.org/D136143
More information about the libc-commits
mailing list