Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hard code dependency resolution bindings for @go_googleapis and WKTs #251

Merged
merged 1 commit into from
Jul 3, 2018

Conversation

jayconrod
Copy link
Contributor

  • proto.csv is a table that lists Well Known .proto files and all
    .proto files in @go_googleapis. For each proto import string, it
    lists the proto_library label, the Go import path, and the
    go_proto_library label. This was generated using an ad hoc Python
    script (not included).
  • Maps are generated from this file and incorporated into the
    dependency resolution logic in the proto and go extensions.

When #12 is implemented, we can index rules in external repositories,
and we won't need this. We need it for now because there's no clear
correspondence between proto and Go import strings and the libraries
that should be included.

Related bazel-contrib/rules_go#1548

@jayconrod jayconrod requested a review from ianthehat July 2, 2018 19:35
* proto.csv is a table that lists Well Known .proto files and all
  .proto files in @go_googleapis. For each proto import string, it
  lists the proto_library label, the Go import path, and the
  go_proto_library label. This was generated using an ad hoc Python
  script (not included).
* Maps are generated from this file and incorporated into the
  dependency resolution logic in the proto and go extensions.

When bazel-contrib#12 is implemented, we can index rules in external repositories,
and we won't need this. We need it for now because there's no clear
correspondence between proto and Go import strings and the libraries
that should be included.

Related bazel-contrib/rules_go#1548
@jayconrod jayconrod force-pushed the googleapis-resolve branch from 002a239 to 17301f1 Compare July 3, 2018 16:35
@jayconrod jayconrod merged commit 79cf429 into bazel-contrib:master Jul 3, 2018
@jayconrod jayconrod deleted the googleapis-resolve branch July 3, 2018 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant