Switch to new_go_repository.

    - Updates targets for go-related code.
    - Removes old custom BUILD files.
    - deps.bzl file cleanup.
    - Remove dangling third_part trunk.
    - Simplify travis to test //...
    - Apply buildifier.

Support proto_library rules in external workspaces.

    - Fixes up the code to shift the execdir into the external workspace,
      then adjust all other paths relative to that.
    - Add prefix attribute to proto_compile rule. This fixes several bugs
      bug related to improper propogation of the go_prefix.
    - Fix descriptor_set path (was previously not offset).
    - Don't offset adjust import paths.

Bump to v0.7.0.

Author: pcj

.travis.yml

@@ -31,12 +31,13 @@ script:
       --spawn_strategy=standalone \
       --genrule_strategy=standalone \
       --local_resources=400,1,1.0 \
-      examples/helloworld/cpp/... \
-      examples/helloworld/go/... \
-      examples/helloworld/java/... \
-      examples/helloworld/closure/... \
-      examples/wkt/... \
-      tests/... \
+      //examples/helloworld/closure/... \
+      //examples/helloworld/cpp/... \
+      //examples/helloworld/go/... \
+      //examples/helloworld/grpc_gateway/... \
+      //examples/helloworld/java/... \
+      //examples/wkt/... \
+      //tests/... \
       $FLAGS
 
 notifications:

Makefile

@@ -1,4 +1,4 @@
-buildify_all:
+fmt:
 	buildifier WORKSPACE
 	find bzl/build_file/ -name '*.BUILD' | xargs buildifier
 	find third_party/ -name BUILD | xargs buildifier

README.md

@@ -90,7 +90,7 @@ language-specific `*_proto_repositories` rule(s):
 git_repository(
   name = "org_pubref_rules_protobuf",
   remote = "https://github.com/pubref/rules_protobuf",
-  tag = "v0.6.4",
+  tag = "v0.7.0",
 )
 
 load("@org_pubref_rules_protobuf//java:rules.bzl", "java_proto_repositories")

WORKSPACE

@@ -34,8 +34,8 @@ closure_repositories()
 
 git_repository(
     name = "io_bazel_rules_dotnet",
-    commit = "b23e796dd0be27f35867590309d79ffe278d4eeb",
-    remote = "https://github.com/pcj/rules_dotnet.git",
+    commit = "9283e4596fbadd7fed237b5c462dfd687c15d301",
+    remote = "https://github.com/bazelbuild/rules_dotnet.git",
 )
 
 load("@io_bazel_rules_dotnet//dotnet:csharp.bzl", "csharp_repositories")
@@ -46,11 +46,6 @@ csharp_repositories(use_local_mono = False)
 # node_proto_library support requires rules_node
 # ================================================================
 
-# local_repository(
-#     name = "org_pubref_rules_node",
-#     path = "/Users/pcj/github/rules_node",
-# )
-
 git_repository(
     name = "org_pubref_rules_node",
     commit = "d93a80ac4920c52da8adccbca66a3118a27018fd",  # Oct 2, 2016
@@ -66,35 +61,99 @@ node_repositories()
 # ================================================================
 
 load("//protobuf:rules.bzl", "proto_repositories")
+
 proto_repositories()
 
 load("//cpp:rules.bzl", "cpp_proto_repositories")
+
 cpp_proto_repositories()
 
 load("//csharp:rules.bzl", "csharp_proto_repositories")
+
 csharp_proto_repositories()
 
 load("//java:rules.bzl", "java_proto_repositories", "nano_proto_repositories")
+
 java_proto_repositories()
+
 nano_proto_repositories()
 
 load("//go:rules.bzl", "go_proto_repositories")
+
 go_proto_repositories()
 
 load("//gogo:rules.bzl", "gogo_proto_repositories")
+
 gogo_proto_repositories()
 
 load("//grpc_gateway:rules.bzl", "grpc_gateway_proto_repositories")
+
 grpc_gateway_proto_repositories()
 
 load("//node:rules.bzl", "node_proto_repositories")
+
 node_proto_repositories()
 
 load("//objc:rules.bzl", "objc_proto_repositories")
+
 objc_proto_repositories()
 
 load("//python:rules.bzl", "py_proto_repositories")
+
 py_proto_repositories()
 
 load("//ruby:rules.bzl", "ruby_proto_repositories")
+
 ruby_proto_repositories()
+
+# ================================================================
+# This is for testing
+# ================================================================
+
+local_repository(
+    name = "org_pubref_rules_protobuf",
+    path = ".",
+)
+
+GOOGLEAPIS_BUILD_FILE = """
+package(default_visibility = ["//visibility:public"])
+
+load("@io_bazel_rules_go//go:def.bzl", "go_prefix")
+go_prefix("github.com/googleapis/googleapis")
+
+load("@org_pubref_rules_protobuf//cpp:rules.bzl", "cc_proto_library")
+load("@org_pubref_rules_protobuf//java:rules.bzl", "java_proto_library")
+load("@org_pubref_rules_protobuf//go:rules.bzl", "go_proto_library")
+
+cc_proto_library(
+    name = "cc_label_proto",
+    protos = [
+        "google/api/label.proto",
+    ],
+    verbose = 0,
+)
+java_proto_library(
+    name = "java_label_proto",
+    protos = [
+        "google/api/label.proto",
+    ],
+    # Neither seem to be necessary, for either 3 langs
+    #imports = ["../../external/com_github_google_protobuf/src", "."],
+    #inputs = ["@com_github_google_protobuf//:well_known_protos"],
+    verbose = 0,
+)
+go_proto_library(
+    name = "go_label_proto",
+    protos = [
+        "google/api/label.proto",
+    ],
+    verbose = 0,
+)
+"""
+
+new_git_repository(
+    name = "com_github_googleapis_googleapis",
+    build_file_content = GOOGLEAPIS_BUILD_FILE,
+    commit = "60c2f9c8d012db35a65539e30fb6364c36f2e96b",
+    remote = "https://github.com/googleapis/googleapis.git",
+)

cpp/deps.bzl

@@ -1,7 +1,3 @@
-# ****************************************************************
-# List of external dependencies
-# ****************************************************************
-
 DEPS = {
 
     # Grpc repo is required by multiple languages but we put it here.

csharp/deps.bzl

@@ -1,7 +1,3 @@
-# ****************************************************************
-# List of external dependencies
-# ****************************************************************
-
 PROTOBUF_BUILD_FILE = """
 load("@io_bazel_rules_dotnet//dotnet:csharp.bzl", "dll_import")
 dll_import(
@@ -48,4 +44,9 @@ DEPS = {
         "build_file_content": GRPC_BUILD_FILE,
     },
 
+    "protoc_gen_grpc_csharp": {
+        "rule": "bind",
+        "actual": "@com_github_grpc_grpc//:grpc_csharp_plugin",
+    },
+
 }

csharp/rules.bzl

@@ -7,6 +7,7 @@ def csharp_proto_repositories(
     omit_cpp_repositories = False,
     lang_deps = DEPS,
     lang_requires = [
+      "protoc_gen_grpc_csharp",
       "nuget_google_protobuf",
       "nuget_grpc",
     ], **kwargs):

examples/helloworld/closure/BUILD

@@ -7,8 +7,8 @@ load("//closure:rules.bzl", "closure_proto_library")
 closure_proto_library(
     name = "protolib",
     protos = [
-        "//examples/proto:protos",
         "//examples/helloworld/proto:protos",
+        "//examples/proto:protos",
     ],
     verbose = 0,
 )
@@ -19,23 +19,22 @@ closure_js_library(
         "greeter.js",
         #"protolib.pb",
     ],
-
     deps = [
         "protolib",
-        "@io_bazel_rules_closure//closure/protobuf:jspb",
         "@io_bazel_rules_closure//closure/library",
+        "@io_bazel_rules_closure//closure/protobuf:jspb",
     ],
 )
 
 closure_js_test(
     name = "greeter_test",
     srcs = ["greeter_test.js"],
-    pedantic = True,
     compilation_level = "SIMPLE",
+    pedantic = True,
     deps = [
         "greeter",
         "protolib",
-        "@io_bazel_rules_closure//closure/protobuf:jspb",
         "@io_bazel_rules_closure//closure/library:testing",
+        "@io_bazel_rules_closure//closure/protobuf:jspb",
     ],
 )

examples/helloworld/go/client/BUILD

@@ -5,14 +5,15 @@ load(
     "//go:rules.bzl",
     GO_GRPC_COMPILE_DEPS = "GRPC_COMPILE_DEPS",
 )
-#load("//gogo:rules.bzl", GOGO_GRPC_COMPILE_DEPS = "GRPC_COMPILE_DEPS")
 
 go_binary(
     name = "client",
     srcs = ["main.go"],
     deps = ["//examples/helloworld/proto:go"] + GO_GRPC_COMPILE_DEPS,
 )
 
+#load("//gogo:rules.bzl", GOGO_GRPC_COMPILE_DEPS = "GRPC_COMPILE_DEPS")
+#
 # To enable this, uncomment and change the importpath to 'gogo' in
 # main.go.
 #

examples/helloworld/grpc_gateway/BUILD

@@ -39,6 +39,6 @@ go_test(
         ":gateway",
         "//examples/helloworld/go/server:greeter",
         "//examples/helloworld/proto:go",
-        "@com_github_golang_protobuf//:jsonpb",
+        "@com_github_golang_protobuf//jsonpb:go_default_library",
     ] + GRPC_GATEWAY_DEPS,
 )