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/types: "user-defined object without package" error while typechecking crypto/tls #28282
Comments
Yes, I just noticed this one, too, locally. Seems new. |
@griesemer I would like to keep the longtest builder green. Should I send a CL ignoring that error for the |
I'm planning to fix it today but won't get to it for a couple of hours.
- Robert
On Fri, Oct 19, 2018, 12:18 PM Alberto Donizetti ***@***.***> wrote:
@griesemer <https://github.com/griesemer> I would like to keep the
longtest builder green. Should I send a CL ignoring that error for the tls
package? Obviously I won't bother if you plan to fix this in the near
future.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#28282 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AIIkTy0iPK_HbuuOcVBsX-GmgdLmyT-cks5umiV2gaJpZM4XxPxU>
.
--
- gri (sent from my phone)
|
Looking into this now. |
Change https://golang.org/cl/143478 mentions this issue: |
This appears to be a subtle issue - won't have time today to track this down. Temporarily disabled part of the failing test per the above CL. |
Keeps the longtest builder green for now. Proper fix to come ASAP. Also, reword an internal comment that could easily be misread. Updates #28282. Change-Id: I8f41c9faa5a3eb638e6204bae3ff374ed49e5177 Reviewed-on: https://go-review.googlesource.com/c/143478 Run-TryBot: Robert Griesemer <gri@golang.org> Reviewed-by: Matthew Dempsky <mdempsky@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
Change https://golang.org/cl/143757 mentions this issue: |
The
go/types
test where it typechecks the whole standard library is currently failing on thecrypto/tls
package:Spotted on the longtest builder.
cc @griesemer
The text was updated successfully, but these errors were encountered: