[libc-commits] [PATCH] D136143: [libc] Add termios.h and the implementation of functions declared in it.

[Michael Jones via Phabricator via libc-commits]

michaelrj accepted this revision.
michaelrj 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);
----------------
sivachandra wrote:
> 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.
SGTM


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136143/new/

https://reviews.llvm.org/D136143