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

Update to spsdk 2.0.0 #486

Closed
PureTryOut opened this issue Dec 8, 2023 · 8 comments · Fixed by #499
Closed

Update to spsdk 2.0.0 #486

PureTryOut opened this issue Dec 8, 2023 · 8 comments · Fixed by #499

Comments

@PureTryOut
Copy link

spsdk 2.0.0 has been released on the 13th of October but the current latest pynitrokey release (0.4.43) isn't compatible with it.

nitropy list
Command line tool to interact with Nitrokey devices 0.4.43
:: 'Nitrokey FIDO2' keys
:: 'Nitrokey Start' keys:
:: 'Nitrokey 3' keys
Critical error:
An unhandled exception occurred
        Exception encountered: ImportError("cannot import name 'StatusCode' from 'spsdk.mboot' (/usr/lib/python3.11/site-packages/spsdk/mboot/__init__.py)")

--------------------------------------------------------------------------------
Critical error occurred, exiting now
@dvzrv
Copy link

dvzrv commented Jan 24, 2024

Packaging this project for Arch Linux, I'd be interested in seeing this as well

@frogamic
Copy link

Maintaining this package in NixOS, would also like to see this.

@daringer
Copy link
Collaborator

daringer commented Jan 29, 2024

couldn't resist so many packager requests - can you please check, if #499 works for you? Especially nk3 update relies most on spsdk.

Tested as thorough as possible, anyways a "remaining-risk disclaimer": This is touching the update process for the lpc55 devices, if this process fails at the wrong time you might end up without a firmware. You would need to open the device to revive it, i.e., boot into bootloader.

@frogamic
Copy link

Thanks @daringer!
This appears to work for me, tested nk3 list, nk3 status and nk3 test, I will try a firmware update when I get home in a few hours so I can use my spare nk3

@frogamic
Copy link

Ah, I noticed you said "update process for the lpc55 devices", my spare nitrokey is an nrf52 based nk3a mini, so not sure if testing it was useful, but it did upgrade successfully

@daringer
Copy link
Collaborator

daringer commented Feb 1, 2024

yes, the nk3mini is based on the nrf52 and does not use spsdk for firmware upgrades.

@frogamic
Copy link

frogamic commented Feb 1, 2024

Ok, I bit the bullet and upgraded my main nk3c nfc, and everything went smoothly.

@daringer
Copy link
Collaborator

daringer commented Feb 1, 2024

uh, nice! always happy if I don't mess things up - including our internal testing this is fine to merge - a new release shouldn't be too far away.

Generally: sorry, for the versioning mess, we'll be working on cleaning this up, feel free to drop further packages with "bad" versions for packaging into issues like @eighthave has done in #494 - we'll try include these into the cleanup efforts and hope to release 0.5 and then 1.0 soon-ish(tm)...

oh and thx for all the packaging work ❤️

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

Successfully merging a pull request may close this issue.

4 participants