Skip to content

Maturin, PyO3, pyo3-asyncio, pyo3-logをアップデート#664

Merged
Hiroshiba merged 20 commits into
VOICEVOX:mainfrom
qryxip:bump-maturin-to-v1.3.1
Oct 29, 2023
Merged

Maturin, PyO3, pyo3-asyncio, pyo3-logをアップデート#664
Hiroshiba merged 20 commits into
VOICEVOX:mainfrom
qryxip:bump-maturin-to-v1.3.1

Conversation

@qryxip
Copy link
Copy Markdown
Member

@qryxip qryxip commented Oct 27, 2023

内容

Maturinを0.13.7から、PyO3/maturin#1811が含まれている1.3.1に上げます。

(追記) pyo3, pyo3-asyncio, pyo3-logもアップデートします。

関連 Issue

その他

CC: @tuna2134

Comment thread .github/workflows/test.yml Outdated
@qryxip qryxip marked this pull request as draft October 28, 2023 11:26
@qryxip qryxip marked this pull request as ready for review October 28, 2023 12:27
@qryxip qryxip changed the title Maturinをv1.3.1に Maturin, PyO3, pyo3-asyncio, pyo3-logをアップデート Oct 28, 2023
@qryxip
Copy link
Copy Markdown
Member Author

qryxip commented Oct 28, 2023

無事CIが通るようになりましたが、ついでにpyo3-logもアップデートします (なんかそのままだと問題が起きそうな気がしたので)
e133286 (#664)

@qryxip
Copy link
Copy Markdown
Member Author

qryxip commented Oct 28, 2023

無事CIが通るようになりましたが、ついでにpyo3-logもアップデート

勘弁してほしい
https://github.com/VOICEVOX/voicevox_core/actions/runs/6676898099/job/18146412030

Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/test.yml Outdated
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.

LGTM!!!!

Comment on lines +283 to +284
poetry run -- cargo build -p voicevox_core_python_api -vv || true
[ -n "$(find ../../target/debug/deps -name 'libopen_jtalk_sys-*.rlib')" ]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

poetry環境でcargo buildする、なるほどです!!

ちなみに|| trueは失敗することがある感じでしょうか? 👀

Copy link
Copy Markdown
Member Author

@qryxip qryxip Oct 29, 2023

Choose a reason for hiding this comment

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

失敗の例としては、これですね。
#664 (comment)

open_jtal-sysのビルド(= Maturinの外でcargo buildする目的)はできますがdylibのリンクには失敗します。そしてdylibのリンクまでは不要です (何故ならそっちはそもそもMaturinがやるべきなので)

Copy link
Copy Markdown
Member

@Hiroshiba Hiroshiba Oct 29, 2023

Choose a reason for hiding this comment

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

へーーーなるほどです。
リンクに失敗するのも不思議感ありました。とりあえずビルドは進みきりそうなので。

ちなみに将来問題が起きたときのためにお聞きしておきたいのですが、maturinがopen_jtalk-sysのビルドに失敗する(open_jtalkのdylibのリンクに失敗する)のは、何が原因っぽい感じなんでしょう 👀
理由はわからないけどmaturinがmacos-12でビルド後のリンク処理に失敗する、みたいな・・・?

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