build: find … -exec … ';'を避ける#98
Conversation
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
find … -exec … ';' でサブコマンド失敗時にジョブが落ちないケースを避け、set -e の対象として失敗を正しく検知できるようにするためのワークフロー修正です。
Changes:
- Linuxクロスコンパイル時の
.so*シンボリックリンク作成をfind -exec … ';'からmapfile + forに変更 - Windowsのコード署名対象DLL列挙・署名実行を
find -exec … ';'からmapfile + forに変更
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. 🤖 Generated with Claude Code |
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
find … -exec … ';' 形式だと -exec の中身が失敗しても find 自体は成功扱いになり、GitHub Actions の bash 実行(-e)でエラー検出できないケースがあるため、-print0 + while read -d '' で逐次実行して失敗を確実に step 失敗へ反映させる変更です。
Changes:
- Linux クロスコンパイル時の
.soシンボリックリンク作成をfind -execからwhileループへ変更 - Windows の DLL コード署名を
find -execからwhileループへ変更
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
内容
;だと中身が失敗しても正常終了するようなので、配列に展開してからちゃんとforで回すことでset -eの対象になるようにする。その他
(記憶が確かなら)私の昨日の業務中に遭遇。忘れないうちにと思いPRを出した次第です。