-
Notifications
You must be signed in to change notification settings - Fork 52
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
Build in workspace #31
Comments
Unfortunately, it's going to be pretty difficult to get cargo-mobile to work with workspaces or even with custom target directories because cargo doesn't really have a reliable option to retrieve the target directory of current project from a crate that's just a dependency, so cargo-mobile right now just guesses that the project uses default I guess it's possible to do something like adding a new key to |
Seems still have problems with workspace. I tried to build android app from workspace member subdirectory and I failed to do it:
Actually the |
Sorry, should have added a comment. After the above comment I realized that I just needed to make a lib.rs file with a |
@alecnotthompson So basically you are not using a workspace anymore? Because I would like to use a workspace and I am running into the problem mentioned by @zeerooth:
Therefore I think this should be reopened? |
No based on what the members said I think there should be a “wontfix” label. I’m still using a workspace but the entry point for each application needs to be in the root which is easily handled with cargo cfg/target attributes |
Cool tool! Found it at just the right time for a project I'm starting.
I am working on a project and am hoping to keep everything in a monorepo. The workspace members are defined with
members = ["library/*"]
and I created the mobile project withcargo mobile new library/mobile
. Everything was set up just fine. Unfortunately I get the following build error:I don't have much time to dig around in the code base to find where to start but if you point me to where the path generation is I can probably fix the rest. Thanks in advance!
The text was updated successfully, but these errors were encountered: