osx-aarch64-cpuをosx-arm64-cpuに改名する
#397
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
pythonパッケージはmaturinが勝手に定義してる感じなんですね！ 以下aarch64かarm64どちらに寄せるかの判断メモです。 CPUアーキテクチャに関して調べてました。めっちゃ奥が深いですね。。。 たぶん世界的な命名として正しい（としたそうな）のはaarch64だろうなと思いました。 ということで、arm64に倒すのが良いのかなと思いました！ |
|
判断メモ続きです。
将来のための検索用ワード |
Merged
qryxip
added a commit
to qryxip/voicevox_core
that referenced
this pull request
Jan 28, 2023
`osx-aarch64-cpu`は`osx-arm64-cpu`に改名される前提にしてしまう。 (VOICEVOX#397)
qwerty2501
pushed a commit
that referenced
this pull request
Jan 28, 2023
* open_jtalk_dicをダウンロードする * `download`のバージョンはセットしない * ZIPの内容を見るためファイルに吐き出していたのを消す * `join_all`を使う * コピペコードを排除 * リファクタ & bug fix * バイナリ自体のarchを使う * `default_value`の指定は`Default`からの`IntoStaticStr`にする * `Accelerator`も同様 * `OptionFuture`を使う * rayonを使う * Fix Cargo.lock * `build_and_deploy`でビルド * Bashを修正 * Minor refactor * `native-tls` → `rustls-tls` * `ProgressStyle`を設定 * PowerShell版の代わりにRust版をテストする * `secrets.GITHUB_TOKEN`を使う * `ProgressStyle`を調整 * [wip] ダウンロード→解凍の手順をひとまとめにする * 2,3段階目のspinnerを用意する * octocrabへのPRはマージされたので切り替える * `Aarch64` → `Arm64` `osx-aarch64-cpu`は`osx-arm64-cpu`に改名される前提にしてしまう。 (#397) * `Vec`のcapacityをちゃんと設定 * `$GITHUB_TOKEN`を使っているところにコメント * `new_pb` → `add_progress_bar`
|
OSとarchの名付けを表にしてみました。
...本当にどうしましょうかこれ... |
|
あー。。osx→macosにしたみはありますね･･･。 |
PickledChair
approved these changes
Jan 29, 2023
|
@qwerty2501 さんにレビューリクエストが行っていますが、多分問題ないと思うのでマージします！（共有です） |
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
osx-aarch64-cpuをosx-arm64-cpuに改名します。関連 Issue
Fixes #393.
その他
Node.js的には
windowsもwin32でosxもdarwinなのでは?と思わなくもないですが、そこまでやる必要は無さそう?やるとしたら今をおいて他にないと思いますが...