Merge pull request #16 from bugsnag/kattrali/windows-support

Author: kattrali

.github/workflows/test-package.yml

@@ -5,30 +5,25 @@ on: [ push, pull_request ]
 jobs:
   test:
 
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}-latest
     defaults:
       run:
         working-directory: 'go/src/github.com/bugsnag/panicwrap' # relative to $GITHUB_WORKSPACE
     strategy:
       fail-fast: false
       matrix:
-        go-version: ['1.7', '1.8', '1.9', '1.10', '1.11', '1.12', '1.13', '1.14', '1.15']
+        os: [ubuntu, windows]
+        go-version: ['1.9', '1.10', '1.11', '1.12', '1.13', '1.14', '1.15']
 
     steps:
     - uses: actions/checkout@v2
       with:
         path: 'go/src/github.com/bugsnag/panicwrap' # relative to $GITHUB_WORKSPACE
-    - name: setup go ${{ matrix.go-version }}
-      run: |
-        curl --silent --location --output gimme https://github.com/travis-ci/gimme/raw/v1.5.4/gimme
-        chmod +x ./gimme
-        eval "$(./gimme ${{ matrix.go-version }})"
-        echo "GOOS=" >> $GITHUB_ENV
-        echo "GOARCH=" >> $GITHUB_ENV
-        echo "GOROOT=$GOROOT" >> $GITHUB_ENV
-        echo "PATH=$PATH" >> $GITHUB_ENV
-        echo "GIMME_ENV=$GIMME_ENV" >> $GITHUB_ENV
-        echo "GOPATH=$GITHUB_WORKSPACE/go" >> $GITHUB_ENV
+    - name: set GOPATH
+      run: echo "GOPATH=$GITHUB_WORKSPACE/go" >> $GITHUB_ENV
+    - uses: actions/setup-go@v2
+      with:
+        go-version: ${{ matrix.go-version }}
     - name: install dependencies
       run: go get -v -d ./...
     - name: run tests

CHANGELOG.md

@@ -1,3 +1,9 @@
+## TBD
+
+### Bug fixes
+
+* Fix windows support by removing undefined syscall
+
 ## 1.3.0 (2021-01-05)
 
 ### Features

dup2.go

@@ -1,4 +1,4 @@
-// +build !arm64
+// +build !arm64,!windows
 
 package panicwrap
 

panicwrap_monitor_test.go

@@ -0,0 +1,25 @@
+// +build !windows
+
+package panicwrap
+
+import (
+	"bytes"
+	"strings"
+	"testing"
+)
+
+func TestPanicWrap_monitor(t *testing.T) {
+
+	stdout := new(bytes.Buffer)
+
+	p := helperProcess("panic-monitor")
+	p.Stdout = stdout
+	//p.Stderr = new(bytes.Buffer)
+	if err := p.Run(); err == nil || err.Error() != "exit status 2" {
+		t.Fatalf("err: %s", err)
+	}
+
+	if !strings.Contains(stdout.String(), "wrapped:") {
+		t.Fatalf("didn't wrap: %#v", stdout.String())
+	}
+}

panicwrap_test.go

@@ -409,22 +409,6 @@ func TestPanicWrap_panicBoundary(t *testing.T) {
 	}
 }
 
-func TestPanicWrap_monitor(t *testing.T) {
-
-	stdout := new(bytes.Buffer)
-
-	p := helperProcess("panic-monitor")
-	p.Stdout = stdout
-	//p.Stderr = new(bytes.Buffer)
-	if err := p.Run(); err == nil || err.Error() != "exit status 2" {
-		t.Fatalf("err: %s", err)
-	}
-
-	if !strings.Contains(stdout.String(), "wrapped:") {
-		t.Fatalf("didn't wrap: %#v", stdout.String())
-	}
-}
-
 func TestWrapped(t *testing.T) {
 	stdout := new(bytes.Buffer)