Minimal CI for OpenPRoT/hubris master

[mynetz]

This PR modifies the upstream hubris CI to build and lint the code.

Following my message here: https://openprot.slack.com/archives/C08QTT3G23X/p1762941476909099

I checked my previous tests and reworked them a bit. The changes needed on the workflow scripts of the upstream project are minimal and shouldn't be a problem.
Testing the build jobs reveals: we need the CI enabled. The code does/did not properly lint (cargo fmt / clippy). Also the license check failed for some code.

The sooner the CI is enabled the better to avoid this in the future.

I did not include the app ast1060-ecdsa-test and ast1060-i2c-scaffold both show errors in clippy that point to unused code and other issues. This is (a) no code we are currently working on (b) its not in scope of this PR to fix every CI issue.

The boards can be added later via build-boards.yml:

  build-openprot:
[...]
        build: [ast1060, ast1060-mctp, ast1060-i2c]
        include:
[...]
          - build: ast1060-i2c
            app_name: ast1060-i2c-scaffold
            app_toml: app/ast1060-i2c-scaffold/app.toml
            image: default

Note:

• The test action is disabled. Some of the code added to the repository over the last month pulls on cortex-m in the build/test which fails on x86 toolchains.
  • https://github.com/OpenPRoT/hubris/actions/workflows/test.yml
• Unneeded actions for now are also disabled:
  • https://github.com/OpenPRoT/hubris/actions/workflows/documentation.yml
  • https://github.com/OpenPRoT/hubris/actions/workflows/release.yml
  • https://github.com/OpenPRoT/hubris/actions/workflows/release-builds.yml
• The added license and the license checked is the default from the hubris project

[mynetz]

I spent some time today to try to figure out why cargo test --verbose --workspace from test.yml fails.

I suspect its pulled in via the aspeed-rust stuff, but I am not sure.

There was one module I found that didn't disable tests even it was for the target toolchain, but it didn't change anything..

$ find . -name \*.toml | xargs grep -l cortex-m | xargs grep -L "test = false"
./drv/openprot-i2c-server/Cargo.toml

Since the goal right now was to get a minimal CI running to have some sort of CI. I skipped it. It should be investigated by someone with more knowledge here. That should then end in a new PR.
I would like to raise an issue but they are disabled for this project.

[mynetz]

@rusty1968 from the commits I gather the code added (and used) by ast1060-ecdsa-test and ast1060-i2c-scaffold is added by you.

The refactor patch touches the files that raised cargo fmt --all -- --check issues. They should be okay now.

Please check cargo xtask clippy app/ast1060-ecdsa-test/app.toml -- --deny warnings and cargo xtask clippy app/ast1060-i2c-scaffold/app.toml -- --deny warnings. They both raise some issue.

I'd open issues for them, but they are disabled in the project.

[FerralCoder]

closing, Hubris usage is deprecated in OpenPRoT