BLD: ビルド時に使用するcythonバージョンを制限する

[sabonerune]

内容

Cython3.0がリリースされたことによってpyopenjtalkのビルドが失敗するようになりました。
一時的な対策としてビルドに使用するバージョンを制限します。

関連 Issue

• ref Build compatibility with Cython 3.0 r9y9/pyopenjtalk#55

[Hiroshiba]

LGTM！！　元の課題が解決されたらバージョン範囲指定解放したいですね！

[Hiroshiba]

VOICEVOX ENGINE側で
poetry add git+https://github.com/VOICEVOX/pyopenjtalk@acd4f02d2af3129382c151590238b9370465e360
したところ、cythonのバージョンは">=0.21.0"で入ってそうでした 😇

diff --git a/poetry.lock b/poetry.lock
index 9cfd457..35e850f 100644
--- a/poetry.lock
+++ b/poetry.lock
@@ -1635,8 +1635,8 @@ files = [
 
 [[package]]
 name = "pyopenjtalk"
-version = "0.3.0"
-description = ""
+version = "0.3.0+acd4f02"
+description = "A python wrapper for OpenJTalk"
 category = "main"
 optional = false
 python-versions = "*"
@@ -1644,6 +1644,7 @@ files = []
 develop = false
 
 [package.dependencies]
+cython = ">=0.21.0"
 numpy = ">=1.20.0"
 six = "*"
 tqdm = "*"
@@ -1657,8 +1658,8 @@ test = ["pytest", "scipy"]
 [package.source]
 type = "git"
 url = "https://github.com/VOICEVOX/pyopenjtalk"
-reference = "827a3fc5c7dda7bbe832c0c69da98e39cc8cb2c3"
-resolved_reference = "827a3fc5c7dda7bbe832c0c69da98e39cc8cb2c3"
+reference = "acd4f02d2af3129382c151590238b9370465e360"
+resolved_reference = "acd4f02d2af3129382c151590238b9370465e360"
 
 [[package]]
 name = "pyrsistent"
@@ -2423,4 +2424,4 @@ testing = ["flake8 (<5)", "func-timeout", "jaraco.functools", "jaraco.itertools"
 [metadata]
 lock-version = "2.0"
 python-versions = "~3.11"
-content-hash = "bee5ebb211d4814a813a618a1325354c79d66b94bf799c8077339738d567e11f"
+content-hash = "04c3ef7fe6818c50549c98a20666cb79f61b895e9eb6ae8910aa8c39b6d74e03"

これは setup.py の方に書かれている内容かもです。

pyopenjtalk/setup.py

Line 21 in acd4f02

min_cython_ver = "0.21.0"

なんで二重管理されてるんだろう & poetryで入るのはどっちなんだろう･･･。

[Hiroshiba]

あ、、無知でした。。
pyproject.tomlの[build-system]のrequiresに書かれているのはビルドに必要なものなんですね。。
ChatGPT君 https://chat.openai.com/share/08720d55-11d8-4274-94f4-5d35e3340614

[sabonerune]

本来は実行時にCythonを使用しないためinstall_requires側のCythonは不要のはずなんですよね…
(開発時には必要なのでextras_requireの方に入れる)

[Hiroshiba]

なるほど～～～～。本家の方になぜ入れているのかとか聞いてみてもいいかもですね！
とりあえずすごくよくわかりました！ありがとうございます！！