-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Windows Support #8
Comments
I'm not sure how membrane automatically moves the shared libs to the root of the dart package, but it'd also be great to get the Windows DLLs moved. |
And I just tried the example on windows 10 x64 and got errors like this
|
The moving of the so/dylib/dll is usually done via something like xtask. Membrane doesn't automatically do the moving for you so you can choose how you want it done as part of your build process. I don't have access to a Windows machine to develop on but if it mostly Just Works™️ and needs the loader addition then I'm happy to accept that. Would you want to open a pull request? So @jc324 is it working for you on Windows but not working at all for @LambdaMan2K? Is there a difference in OS versions or something? |
well I just tried the example and it wont compile do to wrong input somehow. it seems strange if this runs for others :) |
I'm running it on Windows 11, maybe that makes a difference, idk. In #[async_dart(namespace = "accounts")]
pub fn contacts() -> impl Stream<Item = Result<data::Contact, data::Error>> {
futures::stream::iter(vec![Ok(data::Contact::default())])
}
#[async_dart(namespace = "accounts")]
pub async fn contact(user_id: String) -> Result<data::Contact, data::Error> {
Ok(data::Contact {
id: user_id.parse().unwrap(),
..data::Contact::default()
})
} Most of the rest fail with the error mentioned by @LambdaMan2K.
There also seems to be an error on the dart side when calling a rust function that accepts |
Yes, I got those errors too. But using the gnu tool chain worked fine for
me, at least for sending basic data over.
…On Sat, 09 Apr 2022, 6:22 pm LambdaMan2K ***@***.***> wrote:
And I just tried the example on windows 10 x64 and got errors like this
#[async_dart(namespace = "locations")]
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `i64`,
found `i32`
—
Reply to this email directly, view it on GitHub
<#8 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AUD5AA6BC56RAOZVAFIPN33VEGVGBANCNFSM5S654T3Q>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Hi,
Cool library. I would like to suggest windows support, that would be really helpful. So far, I'm just manually patching
loader.dart
with this:Thanks.
The text was updated successfully, but these errors were encountered: