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

Implement hotplug events #64

Open
walac opened this issue Sep 23, 2014 · 13 comments
Open

Implement hotplug events #64

walac opened this issue Sep 23, 2014 · 13 comments

Comments

@walac
Copy link
Member

walac commented Sep 23, 2014

Implement device add/removal events for PyUSB.

@tig3r
Copy link

tig3r commented Oct 16, 2014

I saw that issue: https://github.com/walac/pyusb/issues/22
Is there already a hotplug event notification support for pyusb?

@walac
Copy link
Member Author

walac commented Dec 28, 2015

Not, it doesn't.

@roberthartung
Copy link

FYI: I am working on this.

@roberthartung
Copy link

roberthartung commented Oct 17, 2016

@walac the hotplug callback returns a libusb_device. Where and to which type should it be converted? Should my hotplug implementation to the conversion to Device(_Device(dev, backend))?

@walac
Copy link
Member Author

walac commented Oct 24, 2016

@roberthartung Sorry for the delay, crazy work days. Thank you for working on this, as you already opened a PR, I will discuss design there. I will review it soon. Thanks again.

@roberthartung
Copy link

@walac great! Let me know what you think about the design and I can make changes and make a new pull request if necessary!

@roberthartung
Copy link

@walac Any news on this?

@walac
Copy link
Member Author

walac commented May 30, 2017

@roberthartung I am really sorry for the delay, but I am quite busy in work and personal stuff and could not give PyUSB the love it deserves so far in this year. I hope getting back to it soon.

@roberthartung
Copy link

roberthartung commented Aug 11, 2017

Just uploaded a fix for user_data. See #160

@leileigong
Copy link

how pyUSB hotplug is going?

@roberthartung
Copy link

@leileigong Just merge PR #160 into your fork/branch. It works :-)

@jonasmalacofilho
Copy link
Member

@mcuee I don't see why this issue needs to be pinned.

@mcuee mcuee unpinned this issue Aug 24, 2021
@mcuee
Copy link
Member

mcuee commented Aug 24, 2021

I agree. Just unpin it.

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

No branches or pull requests

6 participants