Skip to content

Conversation

@mchack-work
Copy link
Member

@mchack-work mchack-work commented Jan 23, 2025

Description

Add the hidread program, a simple cross-platform program to read HID events for our debug pipe.

Add extensive documentation on how to build and use hidread and the lshid tool on different platforms.

Closes #12

This PR also updates the older CI setup in the tkey-devtools repo which was in need on some love.

We currently can't include the hidread build in CI, though, so there's an exception for that.

Type of change

Please tick any that are relevant to this PR and remove any that aren't.

  • Feature (non breaking change which adds functionality)
  • Documentation (a change to documentation)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • I have squashed my work to relevant commits and rebased on main for linear history
  • I have updated the documentation where relevant (readme, dev.tillitis.se etc.)

Making place for more commands with different package dependencies.
@mchack-work mchack-work force-pushed the hidread branch 4 times, most recently from 6f51eaf to 7723072 Compare January 24, 2025 09:11
@mchack-work mchack-work requested a review from agren January 24, 2025 10:11
@mchack-work mchack-work force-pushed the hidread branch 2 times, most recently from 7972afd to bce2209 Compare February 17, 2025 14:01
@mchack-work mchack-work marked this pull request as ready for review February 17, 2025 14:02
@agren agren force-pushed the hidread branch 2 times, most recently from b9f3f52 to d42ba99 Compare February 17, 2025 15:01
We rely on the developer or CI to have golangci-lint installed.

However, we run golangci-lint from make target in CI since we have
more than one working directory. For now, though, we skip hidread
since we can't build it easily because of CGO.
- Update to later tkey-builder.

- Use later checkout action.

- Check for SPDX before building!

- Don't try to build hidread in CI because for now because we have too
old Ubuntu in tkey-builder:4. This means we can't install libusb and
libudev, so we remove hidread from all target, too. Leave the
installation of libraries commented out for now.
Remove old stuff from when this was tillitis-key1-apps.
@agren agren self-requested a review February 17, 2025 16:27
@mchack-work mchack-work merged commit f90c45d into main Feb 17, 2025
3 checks passed
@mchack-work mchack-work deleted the hidread branch February 17, 2025 17:59
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.

Research how to access HID Debug on non-Linux

3 participants