Skip to content

Add Windows-specific notes#416

Open
javiermtorres wants to merge 2 commits into
mainfrom
415-windocs
Open

Add Windows-specific notes#416
javiermtorres wants to merge 2 commits into
mainfrom
415-windocs

Conversation

@javiermtorres

Copy link
Copy Markdown
Contributor

Some Windows specific information is needed to build encoderfile on this platform.

@javiermtorres javiermtorres requested review from angpt and besaleli June 15, 2026 08:23

@besaleli besaleli left a comment

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.

one comment, waiting for @angpt to give go ahead!

tokenizer_path: ./models/tokenizer.json
```

> **Note for Windows users:** Since precompiled binaries are not available

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.

link to documentation on building from source

@javiermtorres

Copy link
Copy Markdown
Contributor Author

Thanks @besaleli 🙂


> **Note for Windows users:** Before starting the build process, set `ESAXX_DYNAMIC_LINK=1` to
enable the dynamic linking of the `esaxx-rs` library. Since this library forces static linking,
Mozilla.ai's fork is used instead. This fork only modifies the build process.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What does this mean? Is there a Mozilla.ai library fork?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes. The original https://github.com/Narsil/esaxx-rs library forces static linking, so we created a fork to allow dynamic linking: https://github.com/mozilla-ai/esaxx-rs-dyn-msvc/
Maybe we could try and push the changes to the main repository. Right now this was the only approach we could take.

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