Skip to content

Get rid of lineage-sdk #52

Open
erfanoabdi wants to merge 2 commits into
waydroid:lineage-20from
erfanoabdi:lineage-20
Open

Get rid of lineage-sdk #52
erfanoabdi wants to merge 2 commits into
waydroid:lineage-20from
erfanoabdi:lineage-20

Conversation

@erfanoabdi
Copy link
Copy Markdown
Member

  • Move services to framework repo

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the Waydroid integration by adding the frameworks/waydroid project to the manifest and applying patches to the Android base. These changes transition the codebase from the lineageos.waydroid namespace to id.waydro.waydroid and integrate the Waydroid framework into the build system and SystemServer. Feedback was provided to correct a trace name in SystemServer.java that appears to be a leftover from a different project.

reportWtf("Making " + externalServer + " ready", e);
}

+ t.traceBegin("startLMODroidServices");
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The trace name startLMODroidServices appears to be a leftover from a copy-paste from LMODroid. It should be updated to reflect the Waydroid service being started.

+        t.traceBegin("startWaydroidServices");

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.

1 participant