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.

Sign up for GitHub

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

Jump to bottom

Warning with bazel 0.3.1 #104

Closed
hochhaus opened this issue Sep 16, 2016 · 3 comments
Closed

Warning with bazel 0.3.1 #104

hochhaus opened this issue Sep 16, 2016 · 3 comments
Labels
bug

Comments

@hochhaus
Copy link
Contributor

When using rules_go with bazel 0.3.1 I receive the following WARNING. This isn't a large issue for now, but it will cause an error in future versions.

WARNING: /home/ahochhaus/.cache/bazel/_bazel_ahochhaus/9b0316414c69160d24362f2991b802bb/external/io_bazel_rules_go_toolchain/WORKSPACE:1: Workspace name in /home/ahochhaus/.cache/bazel/_bazel_ahochhaus/9b0316414c69160d24362f2991b802bb/external/io_bazel_rules_go_toolchain/WORKSPACE (@golang_linux_amd64) does not match the name given in the repository's definition (@io_bazel_rules_go_toolchain); this will cause a build error in future versions.

@yugui yugui added the bug label Sep 20, 2016
@pcj pcj mentioned this issue Sep 20, 2016
Merged
@hochhaus
Copy link
Contributor Author

Thanks!

@hochhaus
Copy link
Contributor Author

hochhaus commented Oct 9, 2016 •
edited
Loading

I'm still getting this error with bazel 0.3.2 and 2e6cbd8.

ahochhaus@ahochhaus-pc:~/ll$ bazel version
Build label: 0.3.2
Build target: bazel-out/local-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Fri Oct 7 17:25:10 2016 (1475861110)
Build timestamp: 1475861110
Build timestamp as int: 1475861110

ahochhaus@ahochhaus-pc:~/ll$ bazel build @io_bazel_rules_go//go/toolchain/...
...
WARNING: /home/ahochhaus/.cache/bazel/_bazel_ahochhaus/9b0316414c69160d24362f2991b802bb/external/golang_linux_amd64/WORKSPACE:1: Workspace name in /home/ahochhaus/.cache/bazel/_bazel_ahochhaus/9b0316414c69160d24362f2991b802bb/external/golang_linux_amd64/WORKSPACE (@io_bazel_rules_go_toolchain) does not match the name given in the repository's definition (@golang_linux_amd64); this will cause a build error in future versions.

Can this issue be reopened?

@pmbethe09
Copy link
Contributor

Hmm.
I also get this error.
But if instead I put "golang_linux_amd64" in the written WORKSPACE file I
get:

WARNING:
/home/pbethe/.cache/bazel/_bazel_pbethe/cfa212590cbbbf7eafed3d7d9a8d4855/external/golang_linux_amd64/WORKSPACE:1:
Workspace name in
/home/pbethe/.cache/bazel/_bazel_pbethe/cfa212590cbbbf7eafed3d7d9a8d4855/external/golang_linux_amd64/WORKSPACE
(@io_bazel_rules_go_toolchain) does not match the name given in the
repository's definition (@golang_linux_amd64); this will cause a build
error in future versions.

Apparently a better solution is needed.

On Sat, Oct 8, 2016 at 9:00 PM, Andy Hochhaus notifications@github.com
wrote:

I'm still getting this error with bazel 0.3.2 and 2e6cbd8
2e6cbd8
.

WARNING: /home/ahochhaus/.cache/bazel/_bazel_ahochhaus/9b0316414c69160d24362f2991b802bb/external/golang_linux_amd64/WORKSPACE:1: Workspace name in /home/ahochhaus/.cache/bazel/_bazel_ahochhaus/9b0316414c69160d24362f2991b802bb/external/golang_linux_amd64/WORKSPACE (@io_bazel_rules_go_toolchain) does not match the name given in the repository's definition (@golang_linux_amd64); this will cause a build error in future versions.

Can this issue be reopened?

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
#104 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALF-0lLvwkkaI6GyiUNRstXu8ZwXor0fks5qyDyrgaJpZM4J_M7f
.

@pmbethe09 pmbethe09 reopened this Oct 10, 2016
pcj added a commit to pubref/rules_go that referenced this issue Oct 13, 2016
@pcj
Adds go_root rule.
f0b396f 
- Captures the absolute path of GOROOT at the time of toolchain
  configuration.  This becomes necessary for cross-compile support
  (bazel-contrib#70).

- Symlinks @io_bazel_rules_go_toolchain{src, bin, pkg} to platform
  toolchain paths to avoid workspace name mismatch warnings (closes
  bazel-contrib#104).

- Introduces the RULES_GOROOT environment variable to allow use of a
  pre-existing go toolchain.
@pcj pcj mentioned this issue Oct 13, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@yugui @hochhaus @pmbethe09