Skip to content

build: find … -exec … ';'を避ける#98

Merged
qryxip merged 2 commits intoVOICEVOX:mainfrom
qryxip:pr/build-avoid-find-exec-semicolon
Mar 28, 2026
Merged

build: find … -exec … ';'を避ける#98
qryxip merged 2 commits intoVOICEVOX:mainfrom
qryxip:pr/build-avoid-find-exec-semicolon

Conversation

@qryxip
Copy link
Copy Markdown
Member

@qryxip qryxip commented Mar 13, 2026

内容

;だと中身が失敗しても正常終了するようなので、配列に展開してからforで回すことで ちゃんとset -eの対象になるようにする。

その他

(記憶が確かなら)私の昨日の業務中に遭遇。忘れないうちにと思いPRを出した次第です。

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

AIたちが特に何か問題を見つけなければLGTM!!

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
@qryxip
Copy link
Copy Markdown
Member Author

qryxip commented Mar 13, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

@qryxip qryxip merged commit 9e4ed2b into VOICEVOX:main Mar 28, 2026
6 checks passed
@qryxip qryxip deleted the pr/build-avoid-find-exec-semicolon branch March 28, 2026 01:29
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.

3 participants