リリースに sumomo バイナリを含めるようにする #158

torikizi · 2025-01-31T01:11:46Z

リリース時に sumomo バイナリを含めるように build.yml を修正

This pull request includes changes to the build workflow and documentation. The most important changes involve adding a new step to download and archive example files in the build process and updating the changelog to reflect this addition.

Changes to build workflow:

.github/workflows/build.yml: Added a new step to download and archive example files, including specific handling for macOS and Windows examples.

Documentation updates:

CHANGES.md: Updated the changelog to include the addition of sumomo binaries in the release.

melpon · 2025-01-31T21:16:21Z

.github/workflows/build.yml

+              rm -rf "archive/$d/$d"
+              (cd archive && tar czf "../sumomo_macos_arm64.tar.gz" "$d")
+              echo "sumomo_macos_arm64.tar.gz" >> package_paths.env
+            elif [[ "$d" == *"windows"* ]]; then


プラットフォーム名に対してこういう曖昧な判定をすると、

この部分を見た時に実際に何のプラットフォームに引っかかるのか分からない

後でプラットフォームを増やす時に既存のやつを確認しようと検索してもマッチしないからこの部分を確認できずスルーして意図しないバグを埋め込むことになる

という問題があるので、ワイルドカードを使わずに "$d" == "windows_x86_64" のような書き方をして下さい。

プラットフォームをまとめず一つずつ書くようにしました。

melpon · 2025-01-31T21:17:31Z

.github/workflows/build.yml

+            elif [[ "$d" == *"windows"* ]]; then
+              (cd archive && zip -r "../sumomo_windows_x86_64.zip" "$d")
+              echo "sumomo_windows_x86_64.zip" >> package_paths.env
+            else


ここも同じ問題があるので、意図したプラットフォームを全部列挙して、意図しないプラットフォーム名が来たらエラーになるようにして下さい

バイナリを作りたいプラットフォームを書くようにし、意図しないものは無視するようにしました。

melpon · 2025-01-31T21:23:05Z

.github/workflows/build.yml

+          merge-multiple: false
+      - name: Archive Examples
+        run: |
+          for d in examples_*; do


d という名前が分かりにくいのでちゃんとした名前を指定して下さい。
また、ワイルドカードで列挙すると一覧性が悪くなるので、以下のようにして下さい。

Suggested change

for d in examples_*; do

for platform in \

windows_x86_64 \

macos_arm64 \

ubuntu-20.04_x86_64 \

ubuntu-22.04_x86_64 \

ubuntu-24.04_x86_64 \

ubuntu-24.04_armv8; do

d を使用することをやめました。

melpon · 2025-01-31T21:37:00Z

.github/workflows/build.yml

+              (cd archive && tar czf "../sumomo_macos_arm64.tar.gz" "$d")
+              echo "sumomo_macos_arm64.tar.gz" >> package_paths.env
+            elif [[ "$d" == *"windows"* ]]; then
+              (cd archive && zip -r "../sumomo_windows_x86_64.zip" "$d")


このままだと全部の examples バイナリが含まれてそう

バイナリ格納方法を見直しました。

melpon · 2025-01-31T21:39:35Z

.github/workflows/build.yml

+        run: |
+          for d in examples_*; do
+            mkdir -p "archive/$d"
+            mv "$d" "archive/$d/"


入力となるディレクトリと、そこから sumomo を取り出して配置するディレクトリが同じ場所にあると分かりにくいので、わざわざ mv しなくて良いです。

mv を削除しました

melpon · 2025-01-31T21:41:27Z

.github/workflows/build.yml

+              (cd archive && zip -r "../sumomo_windows_x86_64.zip" "$d")
+              echo "sumomo_windows_x86_64.zip" >> package_paths.env
+            else
+              (cd archive && tar czf "../sumomo_${d#examples_}.tar.gz" "$d")


このままだと全部の examples バイナリが含まれてそう

バイナリ格納方法を見直しました。

torikizi · 2025-02-01T06:41:10Z

レビューありがとうございます。$d やサブシェルなどの処理を見直し、シンプルにしました。
コメントも増やしました。

torikizi · 2025-02-01T10:12:56Z

レビューしていただいているところですが、もう少し見直しをしたいため WIP にします。

torikizi · 2025-02-05T01:50:04Z

.github/workflows/build.yml

@@ -82,20 +82,6 @@ jobs:
        with:
          name: ${{ matrix.name }}.env
          path: _package/${{ matrix.name }}/release/sora.env
-      # Examples のビルド


examples のビルドをわけました。 SDK のビルドの中では行わないようにします。

example 自体のビルドは、ビルドが通ることを確認するためにも必要です。
リリース時にしかビルドしないとリリース時に初めてビルドエラーに気が付くみたいなことになるし、そもそも今はリリース時に sumomo しかビルドしてないから、ほかのサンプルのビルドが通ることを担保できなくなります。

ご指摘のとおりです。
SDK ビルド時の examples のビルドはこれまで通り実施するように戻します。

.github/workflows/build.yml

torikizi · 2025-02-05T01:56:49Z

WIP を解除しました。レビューよろしくお願いします。

.github/workflows/build.yml

torikizi · 2025-02-05T08:30:02Z

変更履歴と build.yml を修正しました。
改めてレビューをお願いします。

.github/workflows/build.yml

melpon · 2025-02-05T11:33:06Z

.github/workflows/build.yml

@@ -82,20 +82,6 @@ jobs:
        with:
          name: ${{ matrix.name }}.env
          path: _package/${{ matrix.name }}/release/sora.env
-      # Examples のビルド


example 自体のビルドは、ビルドが通ることを確認するためにも必要です。
リリース時にしかビルドしないとリリース時に初めてビルドエラーに気が付くみたいなことになるし、そもそも今はリリース時に sumomo しかビルドしてないから、ほかのサンプルのビルドが通ることを担保できなくなります。

melpon · 2025-02-05T11:44:46Z

.github/workflows/build.yml

+        - uses: actions/checkout@v4
+        # Select Xcode 15.4（build-macosと同じバージョンを使う）
+        - name: Select Xcode 15.4
+          run: sudo xcode-select --switch /Applications/Xcode_15.4.app/Contents/Developer


これ必要なのかな…。（入れた経緯がよく分かってない。 shiguredo-webrtc-build/webrtc-build@8d3a781 が大元のコミットっぽい）
変わるたびにバージョン指定をするの大変そうだし、いい感じに設定できるか、設定を無くせると良さそうな気がする。

必要かどうかを見直して対応します。

こちらは導入時に原因となったロジックが削除されていました。削除してビルドが通ることも確認しており、不要と判断して削除します。
差分

.github/workflows/build.yml

melpon · 2025-02-05T21:13:59Z

.github/workflows/build.yml

+      - name: Upload Examples Artifact
+        uses: actions/upload-artifact@v4
+        with:
+          # .tar.gz まで含めないとアップロード時には設定されない


このコメントを書く意味がちょっと分かりませんでした。
name で指定した名前は download-artifact で引っ張ってくる時に利用する名前というだけなので、アーティファクト間で一意であれば何でも良いはずで、アップロードしたアーティファクト名に拡張子が含まれてるかどうかは別に重要ではないです。

元々は SDK ビルド時に作成される examples_ に拡張子が入っていなかったのでわかるようにしたほうが良いと考えていれたコメントでした。
ビルドパッケージを作成する場所でファイルの例も載せているのでここにこのコメントは不要でした。

melpon · 2025-02-05T21:22:04Z

.github/workflows/build.yml

+        - uses: actions/download-artifact@v4
+          with:
+            name: ${{ matrix.example.name }}_${{ matrix.platform.name }}.${{ matrix.platform.file_ext }}
+        # GitHub Releaseを作成し、Examplesをアップロード


リリース自体は既に作られてるので、単にリリースアセットを増やすだけのはず

ご指摘のとおりです。
ここはたんにリリースしているだけになりますのでコメントが間違っていました。
先の # ビルドした Examples のアーティファクトをダウンロード も含めてここは違うコメントか無くすようにします。

CHANGES.md

torikizi · 2025-02-07T01:54:58Z

ご指摘いただいた内容について対応完了しました。CI も通っているのを確認しました。

CHANGES.md

.github/workflows/build.yml

CHANGES.md

torikizi · 2025-02-07T04:01:05Z

動作確認もできました。こちらマージします。
レビューありがとうございました

torikizi added 2 commits January 30, 2025 11:46

リリースに sumomo バイナリを含めるようにする

a8cea51

CHANGES を更新

ba26d43

torikizi requested review from melpon and miosakuma January 31, 2025 01:11

melpon reviewed Jan 31, 2025

View reviewed changes

torikizi added 2 commits February 1, 2025 14:27

sumomo だけを抜き出してアーカイブにする処理を見直し

ca6651e

サブシェルを使わず一つずつ実行する

825b725

torikizi changed the title ~~リリースに sumomo バイナリを含めるようにする~~ [WIP] リリースに sumomo バイナリを含めるようにする Feb 1, 2025

torikizi added 2 commits February 5, 2025 10:45

examples のビルドとリリースを SDK と分ける

6ea4e03

CHANGES を更新

79d0085

torikizi commented Feb 5, 2025

View reviewed changes

torikizi changed the title ~~[WIP] リリースに sumomo バイナリを含めるようにする~~ リリースに sumomo バイナリを含めるようにする Feb 5, 2025

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

torikizi added 2 commits February 5, 2025 16:38

変更履歴を修正

9edd9c5

build.yml の修正を見直し

89d6227

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 5, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

通知の見直し

89d511a

melpon reviewed Feb 5, 2025

View reviewed changes

torikizi added 8 commits February 6, 2025 11:43

コメント修正

df076ba

SDK の examples ビルドを戻す

b33addf

CHANGES を修正

97a4d38

Xcode の指定を外す

f87a35d

Xcode のバージョン指定を外す

b66298f

Ubuntu の examples のビルドを修正

74f312c

Merge branch 'develop' into feature/add-sumomo-release-build-binary

7b5afd5

コメントを少し見直し

d724f90

voluntas reviewed Feb 7, 2025

View reviewed changes

CHANGES.md Outdated Show resolved Hide resolved

voluntas reviewed Feb 7, 2025

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

voluntas reviewed Feb 7, 2025

View reviewed changes

CHANGES.md Outdated Show resolved Hide resolved

voluntas reviewed Feb 7, 2025

View reviewed changes

CHANGES.md Outdated Show resolved Hide resolved

voluntas reviewed Feb 7, 2025

View reviewed changes

CHANGES.md Outdated Show resolved Hide resolved

voluntas reviewed Feb 7, 2025

View reviewed changes

CHANGES.md Outdated Show resolved Hide resolved

voluntas and others added 10 commits February 7, 2025 11:16

不要なコメントを削除

6c5f641

コメントと変更履歴を整理

46e80f9

インデントがおかしいのを修正する

5c19bd6

archive で統一する

38fc98c

name 修正

34de507

strategy を上にする

921a800

strategy を上に持ってくる

875d1cb

不要なコメント削除

b2687ce

コスメ

83a4559

Ubuntu は matrix.name まで指定が必要

378ad70

torikizi merged commit 6f96d8d into develop Feb 7, 2025
14 checks passed

torikizi deleted the feature/add-sumomo-release-build-binary branch February 7, 2025 04:01

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

リリースに sumomo バイナリを含めるようにする #158

リリースに sumomo バイナリを含めるようにする #158

torikizi commented Jan 31, 2025

melpon Jan 31, 2025

torikizi Feb 1, 2025

melpon Jan 31, 2025

torikizi Feb 1, 2025

melpon Jan 31, 2025

torikizi Feb 1, 2025

melpon Jan 31, 2025

torikizi Feb 1, 2025

melpon Jan 31, 2025

torikizi Feb 1, 2025

melpon Jan 31, 2025

torikizi Feb 1, 2025

torikizi commented Feb 1, 2025

torikizi commented Feb 1, 2025

torikizi Feb 5, 2025

melpon Feb 5, 2025

torikizi Feb 6, 2025

torikizi commented Feb 5, 2025

torikizi commented Feb 5, 2025 •

edited

Loading

melpon Feb 5, 2025

melpon Feb 5, 2025

torikizi Feb 6, 2025

torikizi Feb 6, 2025

melpon Feb 5, 2025

torikizi Feb 6, 2025

melpon Feb 5, 2025

torikizi Feb 6, 2025

torikizi commented Feb 7, 2025

torikizi commented Feb 7, 2025

-          for d in examples_*; do
+          for platform in \
+            windows_x86_64 \
+            macos_arm64 \
+            ubuntu-20.04_x86_64 \
+            ubuntu-22.04_x86_64 \
+            ubuntu-24.04_x86_64 \
+            ubuntu-24.04_armv8; do

リリースに sumomo バイナリを含めるようにする #158

リリースに sumomo バイナリを含めるようにする #158

Conversation

torikizi commented Jan 31, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

torikizi commented Feb 1, 2025

torikizi commented Feb 1, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

torikizi commented Feb 5, 2025

torikizi commented Feb 5, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

torikizi commented Feb 7, 2025

torikizi commented Feb 7, 2025

torikizi commented Feb 5, 2025 •

edited

Loading