CI: Fix check for forgotten // +build js constraints in natives.

dmitshur · dmitshur · commit cca79e8e8778 · 2017-07-25T19:28:50.000-04:00
This is a fixup for 9a32def. That commit tried to introduce a CI check for catching when .go files in compiler/natives/src are accidentally missing // +buid js build constraints. However, due to the weird Circle CI setup with symlinks, that CI check wasn't actually going to catch any issues, as proven by test commit a774498 and that it passed that check. However, test commit 7aebc06 showed that relative import path patterns work okay with the Circle CI setup, so use that instead.
diff --git a/circle.yml b/circle.yml
@@ -17,7 +17,7 @@ test:
     - diff -u <(echo -n) <(gofmt -d .)
     - go tool vet *.go                                                   # Go package in root directory.
     - for d in */; do echo $d; done | grep -v tests/ | grep -v third_party/ | xargs go tool vet # All subdirectories except "tests", "third_party".
-    - diff -u <(echo -n) <(go list github.com/gopherjs/gopherjs/compiler/natives/src/...) # All those packages should have // +build js.
+    - diff -u <(echo -n) <(go list ./compiler/natives/src/...) # All those packages should have // +build js.
     - >
       gopherjs test -v --short
       github.com/gopherjs/gopherjs/tests
