thlorenz / rid

Rust integrated Dart framework providing an easy way to build Flutter apps with Rust.
63 stars 4 forks source link

Added linux support. #7

Closed MGlolenstine closed 3 years ago

MGlolenstine commented 3 years ago

Tested on VoidLinux. Updated examples to use rid_api.dart instead of rid_generated.dart. Should close https://github.com/thlorenz/rid/issues/6

thlorenz commented 3 years ago

Thanks for fixing this on Linux ... unfortunately it's tricky to get this right everywhere.

We definitely need a proper CLI tool which would initialize these differently for each platform. It's definitely on the roadmap.

I'm considering cutting wasm support short for now, just do the absolute minimum and then jump onto other things like this one.

Let's keep this PR open as a reference, maybe put it into draft mode for now since we cannot merge as is. However thank you very much for investigating!

thlorenz commented 3 years ago

Alternatively remove the changes that break on OSX and we can at least get all the other fixes merged. Going forward I want to have rid-examples repo be where people go to learn how to use rid. The examples inside the repo are used mostly by rid devs trying to evolve rid itself.

MGlolenstine commented 3 years ago

Alternatively remove the changes that break on OSX and we can at least get all the other fixes merged. Going forward I want to have rid-examples repo be where people go to learn how to use rid. The examples inside the repo are used mostly by rid devs trying to evolve rid itself.

It'd be great to know what breaks on macOS. I don't have access to any Apple devices, so that's kinda out of my scope right now.

I could isolate Linux build scripts for now and just create configurations for Linux alone, as that shouldn't interfere with macOS configs.

thlorenz commented 3 years ago

The llvm-lib: '/usr/lib' change breaks on OSX as I commented. If we remove that for now then the rest can get merged afaik.

MGlolenstine commented 3 years ago

The llvm-lib: '/usr/lib' change breaks on OSX as I commented. If we remove that for now then the rest can get merged afaik.

Cool, if its removal fixes the problem, I'll move it into a separate configuration and make separate build files for Linux and push again.

I'll have to do that tomorrow though (it's almost 1am here).

thlorenz commented 3 years ago

No problem .. take your time and thanks for your help!

MGlolenstine commented 3 years ago

I have pushed an update. I've added sh/linux.yaml and sh/build-linux for compiling on Linux. They work and compile on Linux, and I'd like to request some testing on other platforms. Other platforms should use sh/build to build.

thlorenz commented 3 years ago

Thanks!