Skip to content
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

Renamed events crash the emitter thread on macOS #721

Closed
samschott opened this issue Dec 10, 2020 · 4 comments · Fixed by #722
Closed

Renamed events crash the emitter thread on macOS #721

samschott opened this issue Dec 10, 2020 · 4 comments · Fixed by #722

Comments

@samschott
Copy link
Contributor

Any renamed events crash the emitter thread on macOS with the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/watchdog/observers/fsevents.py", line 97, in queue_events
    and events[i + 1].event_id == event.event_id + 1):
AttributeError: '_watchdog_fsevents.NativeEvent' object has no attribute 'event_id'

The issue, similar to #702, is that the event_id attribute is not defined on the _watchdog_fsevents.NativeEvent class.

I understand that the tests on macOS are currently all skipped and I will submit a PR to fix them. However, until this happens, could you at least test the releases manually by just creating / modifying / renaming a couple of files and seeing if events are emitted? If this is not possible, for instance because you don't have access to the platforms, could you hold off on the next release until the tests are working?

@samschott
Copy link
Contributor Author

Another issue seems to be that those crashes are "silent" (tracebacks don't get printed to the console) which makes them a lot more difficult to notice and report back to you.

@BoboTiG
Copy link
Collaborator

BoboTiG commented Dec 10, 2020

I will hold off for now :)

@samschott
Copy link
Contributor Author

Thanks! And I promise I will contribute to the fixes. It's just difficult for me to keep up with your speed :)

@HEmile
Copy link

HEmile commented Dec 14, 2020

When will this fix be on Pypi?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants