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

go.tools/dashboard/builder: build is broken but not detected by build.golang.org #7372

Closed
davecheney opened this issue Feb 20, 2014 · 5 comments
Milestone

Comments

@davecheney
Copy link
Contributor

What steps will reproduce the problem?

The go.tools repo contains an error

% go build code.google.com/p/go.tools/...
# code.google.com/p/go.tools/dashboard/builder
dashboard/builder/env.go:10: imported and not used: "io"

But the dashboard is not showing an error.

What is the expected output? What do you see instead?

This is because the dashboard builds sub repos as

go get -d code.google.com/p/go.$REPO/...
go test code.google.com/p/go.$REPO/...

because the dashboard/builder package has no tests, no attempt to validate the non test
code is made

Please use labels and text to provide additional information.
@davecheney
Copy link
Contributor Author

Comment 2:

https://golang.org/cl/66800043

Labels changed: added release-go1.3, removed release-none.

Owner changed to @davecheney.

Status changed to Started.

@alexbrainman
Copy link
Member

Comment 3:

Is this related to issue #7108? Should we fix general Problem instead?
Alex

@davecheney
Copy link
Contributor Author

Comment 4:

Yes. I couldn't find that issue but I had a feeling it had been discussed.
This is really a stop gap measure, I'm happy to merge it into issue #7108 if there is an
indication that a more comprehensive solution is desired.

@davecheney
Copy link
Contributor Author

Comment 5:

This issue was updated by revision golang/tools@1c1af4b.

This slipped into the build in revision 4b4229fc616b.
Thanks to @pierredurand87 for noticing the break.
LGTM=bgarcia
R=golang-codereviews, bgarcia
CC=golang-codereviews
https://golang.org/cl/66670043

@rsc
Copy link
Contributor

rsc commented Apr 3, 2014

Comment 6:

Status changed to Duplicate.

Merged into issue #7108.

adg pushed a commit to golang/build that referenced this issue Jan 21, 2015
Update golang/go#7372

This slipped into the build in revision 4b4229fc616b.

Thanks to @pierredurand87 for noticing the break.

LGTM=bgarcia
R=golang-codereviews, bgarcia
CC=golang-codereviews
https://golang.org/cl/66670043
@rsc rsc added this to the Go1.3 milestone Apr 14, 2015
@rsc rsc removed the release-go1.3 label Apr 14, 2015
@golang golang locked and limited conversation to collaborators Jun 25, 2016
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants