Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Commit

Permalink
signal: Removed SIGSTKFLT
Browse files Browse the repository at this point in the history
Fixes #75

According to the man 7 page on Linux the signal is not used. Further, it is not defined on Darwin.
This makes the build compile both on Linux and Darwin.

Signed-off-by: Ricardo Aravena <[email protected]>
  • Loading branch information
raravena80 committed Jun 26, 2018
1 parent 4b80b0f commit 6412558
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 13 deletions.
15 changes: 11 additions & 4 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,15 @@
# SPDX-License-Identifier: Apache-2.0
#

sudo: required
dist: trusty
os:
- linux
- osx

matrix:
include:
- os: linux
sudo: required
dist: trusty

language: go
go_import_path: github.com/kata-containers/proxy
Expand All @@ -17,8 +24,8 @@ before_script:
- ".ci/static-checks.sh"

before_install:
- sudo apt-get update -qq
- sudo apt-get install -y -qq automake
- if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then sudo apt-get update -qq ; fi
- if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then sudo apt-get install -y -qq automake ; fi

install:
- cd ${TRAVIS_BUILD_DIR} && make
Expand Down
17 changes: 8 additions & 9 deletions signals.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,15 +19,14 @@ import (
//
// The value is true if receiving the signal should be fatal.
var handledSignalsMap = map[syscall.Signal]bool{
syscall.SIGABRT: true,
syscall.SIGBUS: true,
syscall.SIGILL: true,
syscall.SIGQUIT: true,
syscall.SIGSEGV: true,
syscall.SIGSTKFLT: true,
syscall.SIGSYS: true,
syscall.SIGTRAP: true,
syscall.SIGUSR1: false,
syscall.SIGABRT: true,
syscall.SIGBUS: true,
syscall.SIGILL: true,
syscall.SIGQUIT: true,
syscall.SIGSEGV: true,
syscall.SIGSYS: true,
syscall.SIGTRAP: true,
syscall.SIGUSR1: false,
}

func handlePanic() {
Expand Down

0 comments on commit 6412558

Please sign in to comment.