skraus-dev / cherryrgb-rs

Cherry RGB Keyboard - Multi platform tool to set RGB LEDs for Cherry keyboards

MIT License

41 stars 3 forks source link

Adapt workflows to #35 #38

Closed felfert closed 1 year ago

felfert commented 1 year ago

Install build dependencies when building the UHID stuff.
Build UHID stuff on native linux only for now.

Description

See my last comment in #35

Fixes workflow issues with #35

Type of change

[x] I have performed a self-review of my own code

felfert commented 1 year ago

BTW: I just noticed, that #39 (a doc-only) PR triggered the CI checks. You probably want to ignore those changes like this:


on:
  push:
    branches: [master]
  pull-request:
    paths-ignore:
      - 'README.md'
      - 'docs/**'

Docs for this feature: here

felfert commented 1 year ago

I dont give up. This is just a fix for the existing CI/CD to handle #35. I will read some more docs and will give it a try later...

felfert commented 1 year ago

This is what I have so far: https://github.com/felfert/cherryrgb-rs/actions/runs/4869978304

Need some more polish, but works nicely. I ditched the cross compilation and build the "foreign" archs within native containers run in qemu. This eleminates the problems with libclang. Also the strip part is unified (no cross binutils required any more). Drawback is longer build times for aarch64:

Matrix build:

Full initial build (no cached container image and no .cargo or target cached): ~25min
With container image and cache hit : ~4min

felfert commented 1 year ago

Closing this. obsoleted by #41