TheDan64 / inkwell

It's a New Kind of Wrapper for Exposing LLVM (Safely)
https://thedan64.github.io/inkwell/
Apache License 2.0
2.38k stars 229 forks source link

Inkwell cannot be built on MacOS #497

Closed BigBadE closed 6 months ago

BigBadE commented 6 months ago

Describe the Bug values/mod.rs line 228 expects section to be a &str, but a conditional compilation block uses format! to modify section, making it a String. This was introduced by #398.

To Reproduce Build on MacOS

Expected Behavior Build success

LLVM Version (please complete the following information):

Desktop (please complete the following information):

Additional Context I'm opening a PR with a fix after a MacOS machine runs a test on the fixed version