Skip to content

Bump magnus from 0.8.2 to 0.8.4 - #132

Closed
thomasklemm wants to merge 1 commit into
IAPark:mainfrom
thomasklemm:magnus-0.8.4
Closed

thomasklemm wants to merge 1 commit into
IAPark:mainfrom
thomasklemm:magnus-0.8.4

Conversation

@thomasklemm

Copy link
Copy Markdown

tiktoken_ruby 0.0.17 and main don't compile against Ruby 4.1 (ruby-head). magnus 0.8.2 fails with:

error[E0063]: missing field `handle_weak_references` in initializer of `rb_data_type_struct__bindgen_ty_1`

magnus 0.8.3 and later handle the new field. This updates Cargo.lock only: magnus 0.8.2 → 0.8.4, plus rb-sys-env 0.2.2 → 0.2.3, which knows Ruby 4.1.

bundle exec rake compile && bundle exec rspec gives 72 examples, 0 failures on Ruby 4.0.7 and on 4.1.0dev (b51db981f6).

magnus 0.8.2 does not compile against Ruby 4.1 (ruby-head), whose
rb_data_type_t gained handle_weak_references.  magnus 0.8.3 and later
cope with it; the update also takes rb-sys-env 0.2.3, which knows
Ruby 4.1.  Cargo.lock only; tests pass on Ruby 4.0.7 and 4.1.0dev.
@gjtorikian

Copy link
Copy Markdown
Collaborator

thanks for this, I just merged #130 which brings magnus up to 0.9.0

@gjtorikian gjtorikian closed this Nov 3, 2026
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.

2 participants