IAPark / tiktoken_ruby

Unofficial ruby binding for tiktoken by way of rust
MIT License
125 stars 26 forks source link

Failed to build on Ventura 13.4.1 M1 and Ruby 2.7.6 #9

Closed teejteej closed 1 year ago

teejteej commented 1 year ago

When trying to build the gem under Ventura 13.4.1 M1 and Ruby 2.7.6, I get the following error:

checking for gcc... yes
checking for g++... yes
checking for libtool -static... no
checking for cargo... yes

current directory: /Users/dev/gems/tiktoken_ruby-0.0.4/ext/tiktoken_ruby
make "DESTDIR=" clean

current directory: /Users/dev/gems/tiktoken_ruby-0.0.4/ext/tiktoken_ruby
make "DESTDIR="
generating target/release/libtiktoken_ruby.dylib (release)
cargo rustc  --manifest-path ./Cargo.toml --target-dir target --lib --profile release -- -C linker=gcc -L native=/Users/dev/rubies/ruby-2.7.6/lib -L native=/opt/homebrew/opt/postgresql@11/lib -L native=/opt/homebrew/opt/libffi/lib -C link-arg=-Wl,-undefined,dynamic_lookup -C
link-arg=-Wl,-multiply_defined,suppress -L native=/opt/homebrew/opt/libyaml/lib -L native=/opt/homebrew/opt/libksba/lib -L native=/opt/homebrew/opt/readline/lib -L native=/opt/homebrew/opt/zlib/lib -L native=/opt/homebrew/opt/openssl@1.1/lib
error: failed to get `tiktoken-rs` as a dependency of package `tiktoken_ruby v0.1.0 (/Users/dev/gems/tiktoken_ruby-0.0.4/ext/tiktoken_ruby)`

Caused by:
  failed to load source for dependency `tiktoken-rs`

Caused by:
  Unable to update https://github.com/IAPark/tiktoken-rs.git#5231fbf4

Caused by:
  failed to parse manifest at `/Users/dev/.cargo/git/checkouts/tiktoken-rs-c5fa179ec3aafdfe/5231fbf/tiktoken-rs/Cargo.toml`

Caused by:
  invalid type: map, expected a sequence for key `package.authors`
make: *** [target/release/libtiktoken_ruby.dylib] Error 101

Any idea what could be the issue?

teejteej commented 1 year ago

Fix by upgrading Rust from 1.5 to 1.7