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

Gazelle: support empty prefixes #912

Merged
merged 1 commit into from
Oct 13, 2017

Conversation

jayconrod
Copy link
Contributor

Gazelle now allows go_prefix to be empty. It must still be set either
on the command line or in a go_prefix rule. When the prefix is empty,
all imports not in the standard library are resolved
locally. Dependencies will not point to the vendor directory or to
external repositories.

  • Gazelle now hard-codes a list of standard packages from the SDK that
    was used to build it.
  • All imports (including standard imports) are now added to
    packages.GoTarget. The resolve package filters out standard
    libraries.

Fixes #870

Gazelle now allows go_prefix to be empty. It must still be set either
on the command line or in a go_prefix rule. When the prefix is empty,
all imports not in the standard library are resolved
locally. Dependencies will not point to the vendor directory or to
external repositories.

* Gazelle now hard-codes a list of standard packages from the SDK that
  was used to build it.
* All imports (including standard imports) are now added to
  packages.GoTarget. The resolve package filters out standard
  libraries.

Fixes bazel-contrib#870
@jayconrod jayconrod merged commit e320e4b into bazel-contrib:master Oct 13, 2017
@jayconrod jayconrod deleted the gazelle-empty-prefix branch October 13, 2017 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gazelle broken when go_prefix is ""
3 participants