patch from issue 697743002

working

The key here was to get everything into a single library.  Our shared library build is the easiest way to do that.  Some light skimming online makes it look like perhaps cgo requires shared libraries, so that may be at play here too.

BUG=skia:

Review URL: https://codereview.chromium.org/698943002

1 files changed, 27 insertions, 19 deletions

diff --git a/experimental/go-skia/skia.go b/experimental/go-skia/skia.go
index 0758c432d1..30730578a7 100644
--- a/experimental/go-skia/skia.go
+++ b/experimental/go-skia/skia.go
@@ -1,30 +1,38 @@
-package skia
+package main
+
+// First, build Skia this way:
+//   ./gyp_skia -Dskia_shared_lib=1 && ninja -C out/Debug
 
 /*
-#cgo CFLAGS: -I../../include/c
-#cgo LDFLAGS: -L ../../out/Debug/
-#cgo LDFLAGS: -lskia_core
-#cgo LDFLAGS: -lskia_effects
-#cgo LDFLAGS: -lskia_images
-#cgo LDFLAGS: -lskia_opts
-#cgo LDFLAGS: -lskia_ports
-#cgo LDFLAGS: -lskia_sfnt
-#cgo LDFLAGS: -lskia_utils
-#cgo LDFLAGS: -lskia_opts_ssse3
-#cgo LDFLAGS: -lskia_opts_sse4
-#cgo LDFLAGS: -lm
-#cgo LDFLAGS: -lstdc++
 #cgo LDFLAGS: -lGL
 #cgo LDFLAGS: -lGLU
+#cgo LDFLAGS: -lX11
+#cgo LDFLAGS: -ldl
+#cgo LDFLAGS: -lfontconfig
+#cgo LDFLAGS: -lfreetype
+#cgo LDFLAGS: -lgif
+#cgo LDFLAGS: -lm
+#cgo LDFLAGS: -lpng
+#cgo LDFLAGS: -lstdc++
+#cgo LDFLAGS: -lz
+
+#cgo LDFLAGS: -L ../../out/Debug/lib
+#cgo LDFLAGS: -Wl,-rpath=../../out/Debug/lib
+#cgo LDFLAGS: -lskia
+
+#cgo CFLAGS: -I../../include/c
 #include "sk_surface.h"
 */
 import "C"
 
-func dummyFunction() {
-	testPaint := C.sk_paint_new()
-	defer func() {
-		sk_paint_delete(testPaint)
-	}()
+import (
+	"fmt"
+)
+
+func main() {
+	p := C.sk_paint_new()
+	defer C.sk_paint_delete(p)
+	fmt.Println("OK!")
 }
 
 // TODO: replace this with an idiomatic interface to Skia.