long standing PR in the hopes one day windows builds work

appveyor.yml

@@ -53,8 +53,9 @@ install:
   - set CXX=g++
   - echo %PATH%
   - echo %GOPATH%
-  - dir C:\mingw64\bin
+  - # dir C:\mingw64\bin
   - go get golang.org/x/net/context
+  - go test -x -v -ldflags "-extldflags '-static'"
 
 build_script:
   - go build -x -ldflags "-extldflags '-static'"

circle.yml

@@ -25,4 +25,4 @@ dependencies:
     - go test -i -race $(go list -f '{{if len .TestGoFiles}}{{.ImportPath}}{{end}}' ./... | grep -v /vendor/)
 test:
   override:
-    - go test -race $(go list -f '{{if len .TestGoFiles}}{{.ImportPath}}{{end}}' ./... | grep -v /vendor/)
+    - go test -v -short -race $(go list -f '{{if len .TestGoFiles}}{{.ImportPath}}{{end}}' ./... | grep -v /vendor/)

context_nowin_test.go

@@ -0,0 +1,41 @@
+// +build !windows
+package libsass
+
+import (
+	"bytes"
+	"testing"
+)
+
+func TestLibsassError(t *testing.T) {
+
+	in := bytes.NewBufferString(`div {
+  color: red(blue, purple);
+}`)
+
+	var out bytes.Buffer
+	ctx := NewContext()
+
+	ctx.Funcs.Add(Func{
+		Sign: "foo()",
+		Fn:   TestCallback,
+		Ctx:  &ctx,
+	})
+	err := ctx.Compile(in, &out)
+
+	if err == nil {
+		t.Error("No error thrown for incorrect arity")
+	}
+
+	if e := "wrong number of arguments (2 for 1) for `red'"; e != ctx.err.Message {
+		t.Errorf("wanted:%s\ngot:%s\n", e, ctx.err.Message)
+	}
+	e := `Error > stdin:2
+wrong number of arguments (2 for 1) for ` + "`" + `red'
+div {
+  color: red(blue, purple);
+}
+`
+	if e != err.Error() {
+		t.Errorf("wanted:\n%s\ngot:\n%s\n", e, err)
+	}
+}

context_test.go

@@ -98,39 +98,6 @@ div {
 
 }
 
-func TestLibsassError(t *testing.T) {
-	in := bytes.NewBufferString(`div {
-  color: red(blue, purple);
-}`)
-
-	var out bytes.Buffer
-	ctx := NewContext()
-
-	ctx.Funcs.Add(Func{
-		Sign: "foo()",
-		Fn:   TestCallback,
-		Ctx:  &ctx,
-	})
-	err := ctx.Compile(in, &out)
-
-	if err == nil {
-		t.Error("No error thrown for incorrect arity")
-	}
-
-	if e := "wrong number of arguments (2 for 1) for `red'"; e != ctx.err.Message {
-		t.Errorf("wanted:%s\ngot:%s\n", e, ctx.err.Message)
-	}
-	e := `Error > stdin:2
-wrong number of arguments (2 for 1) for ` + "`" + `red'
-div {
-  color: red(blue, purple);
-}
-`
-	if e != err.Error() {
-		t.Errorf("wanted:\n%s\ngot:\n%s\n", e, err)
-	}
-}
-
 func ExampleContext_Compile() {
 	in := bytes.NewBufferString(`div {
 			  color: red(blue);

error_test.go

@@ -1,3 +1,4 @@
+// +build !windows
 package libsass
 
 import (

libs/wrap.go

@@ -242,8 +242,9 @@ func SassContextGetOutputString(goctx SassContext) string {
 
 // SassContextGetErrorJSON requests an error in JSON format from libsass
 func SassContextGetErrorJSON(goctx SassContext) string {
-	cstr := C.sass_context_get_error_json(goctx)
+	cstr := C.sass_context_take_error_json(goctx)
 	defer C.free(unsafe.Pointer(cstr))
+	fmt.Println("Error text yo!", C.GoString(cstr))
 	return C.GoString(cstr)
 }
 

libsass-build/sass_context.cpp

@@ -444,8 +444,8 @@ extern "C" {
     if (compiler->state != SASS_COMPILER_CREATED) return -1;
     if (compiler->c_ctx == NULL) return 1;
     if (compiler->cpp_ctx == NULL) return 1;
-    if (compiler->c_ctx->error_status)
-      return compiler->c_ctx->error_status;
+    // if (compiler->c_ctx->error_status)
+    //   return compiler->c_ctx->error_status;
     // parse the context we have set up (file or data)
     compiler->root = sass_parse_block(compiler);
     // success