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
regexp: get rid of the internal/testenv dependency #31350
Comments
Why does it matter? I run tests via |
Sorry I might not be doing the right thing. I'm in my working copy of
I have to do thanks! |
I just change the regexp package in GOROOT without running But in general I don't think we should start removing |
Yes I see your point and unfortunately I don't know enough about the whole context to suggest a proper fix. I just wanted to report this is making life a little harder for novice stdlib contributors :) |
If you set GOROOT explicitly to your working copy root, then that should work, provided your system go binary is a close enough version to your working copy. The best answer is to use the cmd/go binary from the working copy, of course. The main Go repo is a whole that moves forward together, not an a la carte menu. There are plenty of other cross-dependencies besides internal/testenv, and that's OK. |
OK thanks that makes sense. |
regexp/exec_test.go
depends oninternal/testenv
for the sole purpose of doing this in 2 tests:isRaceBuilder := strings.HasSuffix(testenv.Builder(), "-race")
Would it be possible to refactor both tests, possibly moving some of their testcases to the
exec2_test.go
file which has// +build !race
? I'm not sure if they are strictly equivalent.This would make testing patches on the
regexp
module easier.thanks!
The text was updated successfully, but these errors were encountered: