[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