-
Notifications
You must be signed in to change notification settings - Fork 119
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
xcframeworkにmodulemapを追加 #579
xcframeworkにmodulemapを追加 #579
Conversation
mkdir -p "Headers-sim" | ||
cp -v artifact/voicevox_core-x86_64-apple-ios/voicevox_core.h "Headers-sim" | ||
cp -v artifact/voicevox_core-x86_64-apple-ios/module.modulemap "Headers-sim" | ||
mkdir -p "Headers-aarch64" | ||
cp -v artifact/voicevox_core-aarch64-apple-ios/voicevox_core.h "Headers-aarch64" | ||
cp -v artifact/voicevox_core-aarch64-apple-ios/module.modulemap "Headers-aarch64" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-headers
オプションがディレクトリ指定のみだったので、Headersに入れるファイルを一度ディレクトリに入れ、それを指定しています。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
なるほどです!
ちなみに知見として知りたいのですが、最初が大文字なのってそういう制約だったりされますか 👀
(公式?は特に指定なさそう感・・・? https://developer.apple.com/documentation/xcode/creating-a-multi-platform-binary-framework-bundle )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
この一時的に作成しているディレクトリの名前は大文字小文字関係ないです。
xcframeworkにした際に、xcodebuildが-headers
に指定したディレクトリの中身をデフォルトでHeaders
というディレクトリの中にコピーするので、simlatorとarrch64用のそれぞれのHeadersにコピーされるディレクトリという意味でHeaders-sim
とHeaders-arrch64
としました。
voicevox_core.xcframework
├── Info.plist
├── ios-arm64
│ ├── Headers
│ │ ├── voicevox_core.h
│ │ └── module.modulemap
│ └── libvoicevox_core.dylib
└── ios-arm64_x86_64-simulator
├── Headers
│ ├── voicevox_core.h
│ └── module.modulemap
└── libvoicevox_core.dylib
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
なるほどです!
他のとこに比べて命名規則が変わっているのが若干だけ気になったのですが、コピー先のディレクトリが文字始まりなら、まあ問題ではないのでそれでもいいかなと思いました!
大変お忙しいところ、コメントいただきありがとうございます🙇 |
Co-authored-by: Hiroshiba <[email protected]>
確認ありがとうございます! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!!
マージ後に1回テストでビルドを回してみたいと思います!
内容
関連 Issue
close #578
その他