VOICEVOX / open_jtalk-rs

BSD 3-Clause "New" or "Revised" License
10 stars 12 forks source link

aarch64用ビルドを追加 #3

Closed haru3me closed 1 year ago

haru3me commented 1 year ago

aarch64用のビルドを追加します.

i8ポインタが環境依存のようなのでc_charに置換えを行いました.

変更後のプロジェクトを組み込んだvoicevox_coreのビルドは通りましたが,CIのrust-lintが失敗しているのが気になっています.

   Compiling rustc_version v0.4.0
   Compiling open_jtalk-sys v0.16.111 (/home/runner/work/open_jtalk-rs/open_jtalk-rs/crates/open_jtalk-sys)
error: the borrowed expression implements the required traits
  --> crates/open_jtalk-sys/build.rs:48:35
   |
48 |     let paths = std::fs::read_dir(&include_dir).unwrap();
   |                                   ^^^^^^^^^^^^ help: change this to: `include_dir`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
   = note: `-D clippy::needless-borrow` implied by `-D clippy::all`

error: could not compile `open_jtalk-sys` due to previous error
warning: build failed, waiting for other jobs to finish...
Error: Process completed with exit code 101.

よろしくお願いします.

qryxip commented 1 year ago

CIのrust-lintが失敗しているのが気になっています.

4 を出しました。

haru3me commented 1 year ago

お手数おかけして申し訳ないです。ありがとうございます。

qwerty2501 commented 1 year ago

i8ポインタが環境依存のようなので

環境によってcharの実体がi8かu8かになるからでしょうか?

qryxip commented 1 year ago

だと思います。aarch64のLinuxだとu8になるようなので。

https://github.com/rust-lang/rust/blob/1.66.0/library/core/src/ffi/mod.rs#L104-L157

haru3me commented 1 year ago

環境によってcharの実体がi8かu8かになるからでしょうか?

はい bindings自体は問題なく生成できるのですが、ライブラリの実態に適合しないようでvoicevox_coreでのビルド時にエラーが出てしまいました。

qryxip commented 1 year ago

4 がマージされたので、こちらを取り込んで頂ければCIが通ると思います。

haru3me commented 1 year ago

ごめんなさい フォーマットを忘れていました.手元のCIで通ったのでこれで大丈夫だと思います.