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

Update build check CI to follow the Raspberry Pi official cross compile procedure #86

Merged
merged 9 commits into from
Sep 2, 2024

Conversation

ShotaAk
Copy link
Contributor

@ShotaAk ShotaAk commented Aug 30, 2024

What does this implement/fix?

Raspberry Pi 公式のkernelクロスコンパイル手順に従うように、workflowファイルを変更します。

公式手順:https://www.raspberrypi.com/documentation/computers/linux_kernel.html

なぜ変更するのか?

現在のCIに使用しているRaspberry Pi GCC Toolchains Filesにて、
最新のRaspberry Pi OS(Debian version: 12 (bookworm))向けのtoolがリリースされていないため、
代わりの手段を検討しています。

Raspberry Pi 公式の手順でビルドチェックができたため、本PRを作成しました。

その他変更点

  • カーネルのバージョン指定をコミットハッシュではなくブランチ指定に変更します
    • メンテナンス工数を削減するためです
    • コミット基準でビルドチェックをしたい状況がほとんど無いためです
  • カーネルコンフィグの上書きを省略します
    • メンテナンス工数を削減するためです

Does this close any currently open issues?

いいえ

How has this been tested?

nektos / actを使って、ローカル環境でモジュールがビルドできることを確認しています。

Checklists

@ShotaAk ShotaAk added the Type: CI Changes to CI configuration files and scripts label Aug 30, 2024
Copy link
Contributor

@KuraZuzu KuraZuzu left a comment

Choose a reason for hiding this comment

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

今回のCI変更によって、.test/配下のファイルのうち、lint.sh以外は不要になったのではないかと感じました。もしそうであれば、これらのファイルは削除すべきだと考えます。

Copy link
Contributor

@KuraZuzu KuraZuzu left a comment

Choose a reason for hiding this comment

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

LGTMです。.test内のbinまで不要なディレクトリとして扱う間違った指摘をしていましたが、そちらにもご対応いただきありがとうございました。

@KuraZuzu KuraZuzu merged commit f5dfa98 into master Sep 2, 2024
4 checks passed
@KuraZuzu KuraZuzu deleted the new_build_test_ci branch September 2, 2024 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: CI Changes to CI configuration files and scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants