-
Notifications
You must be signed in to change notification settings - Fork 230
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
Updating from 0.25.3 to 0.26.x or 0.27.x fails for interface #2127
Comments
I assume you mean 0.25, etc. (there's no |
I cannot reproduce this locally. Code inline below. If you can providing the failing code in a git repository would help us debug this further.
use std::time::SystemTime;
#[allow(non_camel_case_types)]
pub enum ExampleEnum {
v1, v2
}
pub struct Glucose {
raw: f32,
timestamp: SystemTime,
local_timestamp: SystemTime,
}
pub fn hello_world() -> String {
String::from("hello world")
}
pub struct TodoList;
impl TodoList {
fn new() -> Self {
TodoList {
}
}
}
uniffi::include_scaffolding!("uniffi-bug-test"); UDL: namespace uniffi_bug_test {
/// Hello Rust
string hello_world();
};
/// Example Enum
enum ExampleEnum {
"v1",
"v2",
};
/// Glucose
dictionary Glucose {
f32 raw;
timestamp timestamp;
timestamp local_timestamp;
};
/// TodoList
interface TodoList {
constructor();
}; |
@badboy thanks for looking into it, I'll try building a repo that reproduces the issue. I failed to mention some information:
|
@badboy Indeed, renaming our local core lib to anything else works... |
There are some places we use |
@mhammond That was my thought as well, I've unblocked us by renaming our current library, but I bet lots of repos would have a "core" package internal to them, would be nice to prioritize this issue |
Ah, somehow that is the one instance I overlooked. Yes, changing that to
I kinda doubt that many people name their crate |
Env:
nightly-aarch64-apple-darwin - Up to date : 1.80.0-nightly (867900499 2024-05-23)
uniffi 0.27.2
The following works
Adding the following fails:
The error is:
This is the content of example.uniffi.rs:69:1
The text was updated successfully, but these errors were encountered: