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

Speculos with HID + BUSB #9

Open
rithvikvibhu opened this issue Jul 14, 2021 · 3 comments
Open

Speculos with HID + BUSB #9

rithvikvibhu opened this issue Jul 14, 2021 · 3 comments

Comments

@rithvikvibhu
Copy link
Member

With busb removed, it's no longer possible to use the emulator. Would be nice if it's added back, but not a high priority as it only helps a few people without devices and during dev.

@nodech
Copy link
Contributor

nodech commented Jul 14, 2021

I believe we can include both for the hsd-ledger library, probably as optional dependencies. (E.g. WebUSB can only be supported by busb)

This is the current situation of the supported platforms

Linux MacOS Windows WebUSB Speculos
HID + + + - -
BUSB + + -* + +
  • BUSB (node-usb) - needs drivers for windows

Only thing that needs slight adjustments is API compatibility, after compatibilities are fixed it should be possible to do optional thing.

@pinheadmz
Copy link
Member

We could add the speculos feature on top of HID as well, the way it worked with busb is just by changing the require() statement. So the speculos.js module might just need to be tweaked a bit to match the HID API, which isn't too different from the busb API anyway.

@nodech
Copy link
Contributor

nodech commented Jul 15, 2021

I have not looked into speculos much, but I believe we could even move that out of busb and make separate device, like busb and hid. This could also allow us to run tests for lower layer easier (apdu/an inner workings). - If it can be compiled on CI even better.

@rithvikvibhu rithvikvibhu changed the title Speculos with HID Speculos with HID + BUSB Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants