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

flake.nix + qemu.mk : add working qemu-canokey usable from all qemu boards by default #1671

Merged
merged 1 commit into from
May 13, 2024

Conversation

tlaurion
Copy link
Collaborator

@tlaurion tlaurion commented May 12, 2024

  • flake.nix: add canokey-qemu lib, derivate qemu on tope of it and have qemu_kvm depend on qemu derivative
  • targets/qemu.mk: modified to had canokey support by default if no "USB_TOKEN=" specified on make run call
  • CircleCI: base docker image pull on v0.1.6 containing the newly added derivatives

Fixes #1670
Closes #1207


Finally, bye errors of qemu TCG on top of qemu on top of qubesos with usb error -32.

It's alive!!!
2024-05-12-132729


Someone.... wants to implement reverse HOTP inside of canokey-qemu???

…oards by default

flake.nix: add canokey-qemu lib, derivate qemu on tope of it and have qemu_kvm depend on qemu derivative
targets/qemu.mk: modified to had canokey support by default if no "USB_TOKEN=" specified on make run call

CircleCI: base docker image pull on v0.1.6 containing the newly added derivatives
Signed-off-by: Thierry Laurion <insurgo@riseup.net>
@tlaurion
Copy link
Collaborator Author

tlaurion commented May 12, 2024

To test and replicate:

docker run -e DISPLAY=$DISPLAY --network host --rm -ti -v $(pwd):$(pwd) -w $(pwd) tlaurion/heads-dev-env:latest -- make BOARD=qemu-coreboot-fbwhiptail-tpm2
docker run -e DISPLAY=$DISPLAY --network host --rm -ti -v $(pwd):$(pwd) -w $(pwd) tlaurion/heads-dev-env:latest -- make BOARD=qemu-coreboot-fbwhiptail-tpm2 run

Follow:

@tlaurion
Copy link
Collaborator Author

@JonathonHall-Purism quick review, doesn't change much but canokey being usable if no usb security token being passed in make run call

Copy link
Collaborator

@JonathonHall-Purism JonathonHall-Purism left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine 👍

@tlaurion tlaurion merged commit 70b3272 into linuxboot:master May 13, 2024
39 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
2 participants