Skip to content

go/types: std lib tests don't consider +build tags #10370

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
Closed
griesemer opened this issue Apr 8, 2015 · 5 comments
Closed

go/types: std lib tests don't consider +build tags #10370

griesemer opened this issue Apr 8, 2015 · 5 comments
Labels
FrozenDueToAge
Milestone
Go1.12

Comments

@griesemer
Copy link
Contributor

Some of the std lib tests (e.g. solaris-amd64) are failing because +build tags are ignored. The tests should consider the build tags in place for the test binary (because it depends on gcimporter which sees the files as built by gc for the current platform).
@griesemer griesemer self-assigned this Apr 8, 2015
@griesemer griesemer added this to the Go1.5Maybe milestone Apr 8, 2015
@rsc rsc changed the title go/types (std repo): std lib tests don't consider +build tags go/types: std lib tests don't consider +build tags Apr 26, 2015
@griesemer griesemer modified the milestones: Go1.6, Go1.5Maybe Jun 23, 2015
@griesemer
Copy link
Contributor Author

This is not super-important. Could be delayed to 1.7 or later w/o much repercussions.

@minux
Copy link
Member

minux commented Sep 24, 2015

could we also consider exclude $GOROOT/src/*.go?
I usually place short repro files there when fixing bugs
because it's the most convenient place to place
test files. (see also https://golang.org/cl/14769 for a
similar issue wrt to go/build-ignored directories.)

@rsc rsc modified the milestones: Unplanned, Go1.6 Dec 11, 2015
@griesemer
Copy link
Contributor Author

We already exclude files in $GOROOT/src. And go/importer (go/internal/gcimporter) is using go/build. I don't believe this is still a valid issue. Closing.

@griesemer
Copy link
Contributor Author

I closed this prematurely. The issue is that go/types/stdlib_test.go's testTestDir function doesn't consider build tags I believe.

@griesemer griesemer reopened this Nov 1, 2017
@griesemer griesemer modified the milestones: Unplanned, Go1.10 Nov 1, 2017
@rsc rsc modified the milestones: Go1.10, Go1.11 Nov 22, 2017
@griesemer griesemer modified the milestones: Go1.11, Go1.12 Jun 27, 2018
@gopherbot
Copy link
Contributor

Change https://golang.org/cl/144457 mentions this issue: go/types: automatically ignore $GOROOT/test files that contain build tags

@golang golang locked and limited conversation to collaborators Oct 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Go1.12
Development

No branches or pull requests
4 participants
@rsc @minux @griesemer @gopherbot