libui-rs icon indicating copy to clipboard operation
libui-rs copied to clipboard

Update build.rs to handle CMake build system

Open talklittle opened this issue 8 years ago • 7 comments

Upstream libui has changed to a CMake build system.

ui-sys/build.rs should be updated to handle it.

talklittle avatar Jun 05 '16 06:06 talklittle

It looks like @Machtan already has a working build.rs that handles CMake!

https://github.com/Machtan/libui-rs/blob/3f5f79dce7701b38783763da14f97f14f2b140c0/ui-sys/build.rs

@Machtan, would you be willing to submit a PR for that?

talklittle avatar Jun 05 '16 06:06 talklittle

Sure. The version of the libui submodule referenced might have to be changed, though. At least I get a crash on the controlgallery example and weird artifacts on the rust logo one.

ghost avatar Jun 05 '16 07:06 ghost

The pull request is made. (I got it to use alpha3 too, but the examples are still unusable on Mac)

ghost avatar Jun 05 '16 07:06 ghost

Cool thanks! Actually I believe alpha3 is just before the CMake changes upstream. I guess your build.rs is robust enough to handle both old and new build systems.

Anyway, I made pull request #3 to fix the controlgallery.rs example, tested on Mac, but haven't attempted to fix the Rust logo artifacts.

talklittle avatar Jun 05 '16 08:06 talklittle

I just realized the thing about alpha3, so now it's gone again. Should I update to include the ~10 CMake related commits made since yesterday?

ghost avatar Jun 05 '16 08:06 ghost

Sure, I think that would be really cool. It's all up to you though. I'm not a maintainer of this library, so my suggestions don't necessarily matter much. :)

talklittle avatar Jun 05 '16 08:06 talklittle

I have a pull request up now which I think solves this issue. #14

zentner-kyle avatar Apr 04 '17 04:04 zentner-kyle