-
Notifications
You must be signed in to change notification settings - Fork 13k
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
make check-build-lexer-verifier
fails currently
#22379
Comments
I am pretty sure that it doesn't actually work, and that all of this has just rotted. |
cc me (i think there is something going wrong where only some host environments will actually try to run the steps associated with this target, and our buildbots are not in that set. But I do know that I hit this problem myself when I try to emulate the buildbots and run |
…, r=alexcrichton I know this is the most trivial thing since it's *just* the model lexer, not the real lexer, but [it is simpler to read](rust-lang#15883 (comment)) and it'd be great if it was up to date but [it's been rotting](rust-lang#22379) and this is a tiny bit of that. Thanks!!!!!!
…, r=alexcrichton I know this is the most trivial thing since it's *just* the model lexer, not the real lexer, but [it is simpler to read](rust-lang#15883 (comment)) and it'd be great if it was up to date but [it's been rotting](rust-lang#22379) and this is a tiny bit of that. Thanks!!!!!!
Triage: on my machine:
So, it seems like this has been fixed, at least on my machine. Though as I said above, it's not like this is seeing any active development, and I don't even know if it's truly working, just that it returns zero. I'm not sure if this should be left open or closed, frankly... |
Make's now gone, so closing. |
I'm surprised this is not causing issues because the Makefile claims that this target should get run by Bors everytime upon check-in
The text was updated successfully, but these errors were encountered: