Skip to content
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

waydroid: initial implementation (Updated for the latest version of waydroid) #183

Merged
merged 3 commits into from
Nov 1, 2022

Conversation

vroad
Copy link
Contributor

@vroad vroad commented Apr 16, 2022

I updated @danielfullmer 's PR (#114) to build with the latest version of waydroid. I tested the image on waydroid. It successfully booted, and I haven't found problems specific to my image so far.

This PR contains changes that might affect other flavors:

  • I added wget to nix-shell because waydroid's manifest generator script relys on it.
  • I switched from patch to git apply, because patch doesn't support applying binary patches. Waydroid requires applying binary patches to source code to build the image.

danielfullmer and others added 2 commits April 16, 2022 13:14
Successfully builds, but untested.
Patching script now uses git command for applying binary patches.
(patch command does not currently support binary patches, unfortunately)
@danielfullmer
Copy link
Collaborator

Could you add a short section to docs/src/modules/flavors.md (similar to Anbox section) describing the support level as "experimental"?

@vroad
Copy link
Contributor Author

vroad commented Apr 17, 2022

Could you add a short section to docs/src/modules/flavors.md (similar to Anbox section) describing the support level as "experimental"?

I added a section, though mostly just a copy and paste of anbox section. Feel free to improve on it.

@danielfullmer danielfullmer merged commit 6d73adf into nix-community:master Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants