-
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
internal compiler error #23133
Comments
I actually encountered an ICE within this library when attempting to change this: fn unwatch(&mut self, path: &Path) -> Result<(), Error> {
// FIXME:
// once https://github.com/rust-lang/rust/pull/22351 gets merged,
// just use a &Path
match self.watches.remove(&path.to_path_buf()) {
None => Err(Error::WatchNotFound), to this: fn unwatch(&mut self, path: &Path) -> Result<(), Error> {
// FIXME:
// once https://github.com/rust-lang/rust/pull/22351 gets merged,
// just use a &Path
match self.watches.remove(path) {
None => Err(Error::WatchNotFound), This caused an ICE, so I decided not to make that change yet. Updating to the nightly in the first post from |
I just tested it and it works with the nightly from 2015-03-04, so it's the most recent nightly 2015-03-05 which I believe causes the problem. |
This looks just like #22432, so I'm surprised that the nightly from 3/4 works. |
Yeah that's weird. It works with this:
|
One of my crates broke in the latest nightly with this bug when I use |
This should have been fixed by PR #23074. If you find the time, please confirm with a current nightly. |
Works for me. |
PR #23074 resolves for me. |
Thanks for the feedback. Let's wait some more if the OP has a comment. Otherwise I'll close this tomorrow. |
Yeah I'll give this a try later today. On Thursday, March 12, 2015, Michael Woerister [email protected]
|
Just confirmed that this is resolved. Thanks!! |
Excellent. Thanks everyone! |
Building the latest version of the
notify
crate (1.2.0) yields an internal compiler error.output:
The text was updated successfully, but these errors were encountered: