Fixed use of TIOCSTI on Linux 6.2 or higher

gansm · Sep 26, 2024 · 8a35abe · 8a35abe
1 parent ae2ee06
commit 8a35abe
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
@@ -46,11 +46,9 @@ jobs:
 
     - name: Build with with gcov
       run: |
-        ls -al /proc/sys/dev/tty/legacy_tiocsti
-        echo "1" > /proc/sys/dev/tty/legacy_tiocsti 2>/dev/null || true # Required for Linux 6.2.0 or later
+        sudo sysctl -w dev.tty.legacy_tiocsti=1  # Required for Linux 6.2.0 or later
         echo "/proc/sys/dev/tty/legacy_tiocsti"
         cat /proc/sys/dev/tty/legacy_tiocsti
-        sysctl -w dev.tty.legacy_tiocsti=1
         script -q -e -c './build.sh coverage'
 
     - name: Generate lcov.info and publish to coveralls.io

diff --git a/.github/workflows/coverity-scan.yml b/.github/workflows/coverity-scan.yml
@@ -76,9 +76,7 @@ jobs:
 
     - name: Make unit test
       run: |
-	setcap cap_sys_admin+ep test/.libs/eventloop_monitor_test
-	getcap test/.libs/eventloop_monitor_test
-        echo "1" > /proc/sys/dev/tty/legacy_tiocsti 2>/dev/null || true # Required for Linux 6.2 or later
+        sudo sysctl -w dev.tty.legacy_tiocsti=1  # Required for Linux 6.2.0 or later
         script -q -e -c 'make check' && RET=$? || RET=$?
         cat test/*.log || echo
         echo "Exit status: $RET"