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

new url for libzbar; altered config for seedsigner service #637

Merged
merged 1 commit into from
Dec 22, 2024

Conversation

jdlcdl
Copy link

@jdlcdl jdlcdl commented Dec 21, 2024

Description

While recently rebuilding my manual-build development raspi-os sdcard image, I ran into two problems:

This pull request is categorized as:

  • New feature
  • Bug fix
  • Code refactor
  • Documentation
  • Other

Checklist

  • I’ve run pytest and made sure all unit tests pass before submitting the PR

If you modified or added functionality/workflow, did you add new unit tests?

  • No, I’m a fool
  • Yes
  • N/A

I have tested this PR on the following platforms/os:

Note: Keep your changes limited in scope; if you uncover other issues or improvements along the way, ideally submit those as a separate PR. The more complicated the PR the harder to review, test, and merge.

@kdmukai
Copy link
Contributor

kdmukai commented Dec 21, 2024

Download and install of updated libzbar was successful. Was able to scan in a SeedQR so presumably zbar is indeed able to do its thing as expected.

Updated seedsigner.service definition and it looks like it's working fine as well.

Tested ACK.


Aside: discovered that macOS Ventura has knocked out the internet-over-usb relay option that I had been using on my dev Pi Zero to reach the outside world (it can still do ssh and rsync kind of stuff back and forth with the Macbook Pro).

@newtonick
Copy link
Collaborator

ACK

@newtonick newtonick merged commit cfc22b0 into SeedSigner:dev Dec 22, 2024
2 checks passed
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 this pull request may close these issues.

3 participants