Skip to content

Assume a value for AUTOCXX_RS #178

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

Closed
adetaylor opened this issue Jan 8, 2021 · 1 comment · Fixed by #181
Closed

Assume a value for AUTOCXX_RS #178

adetaylor opened this issue Jan 8, 2021 · 1 comment · Fixed by #181
Labels
good first issue Good for newcomers

Comments

@adetaylor
Copy link
Collaborator

Following #171 our macro now simply include!s a file generated in an earlier codegen phase.

It requires AUTOCXX_RS to be set to educate it where to find such files, but in a cargo context it should be able to assume a default.

It seems quite likely that rust-analyzer and friends will be able to parse and understand the FFI if we manage to do htis.

@adetaylor
Copy link
Collaborator Author

Fixed, but didn't perfectly do the trick. rust-analyzer can now find the generated .rs file and that's terrific; but it doesn't attempt to expand the #[cxx::bridge] macro inside it. Ideally we'd take this forward by providing a minimal repro to rust-analyzer, and perhaps I'll do that in future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant