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

Trigger Zygisk modules service.sh in parallel #91

Merged
merged 2 commits into from
Nov 24, 2024

Conversation

JingMatrix
Copy link
Contributor

Changes

Change the way ReZygisk executing service.sh of zygisk modules.

Why

This will solve issue #90 and JingMatrix/LSPosed#62 so that LSPosed can run flawlessly with ReZygisk on Magisk.

See explanation in JingMatrix/LSPosed#62 (comment)_

Checkmarks

  • The modified functions have been tested.
  • Used the same indentation as the rest of the project.
  • Updated documentation (changelog).

Additional information

If you have any additional information, write it here

Copy link

@kevin01523 kevin01523 left a comment

Choose a reason for hiding this comment

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

Hmm looks fine to me but needs to be tested by @ThePedroo and by different users i guess

For scripts in late_start service mode, they should be executed in
parallel, as explained in the official docs:

https://github.com/topjohnwu/Magisk/blob/master/docs/guides.md#boot-scripts
GitHub CI is broken due to outdated gradle action
@JingMatrix
Copy link
Contributor Author

I have to update the gradle ci since it was broken in my repo, see evidence here: https://github.com/JingMatrix/ReZygisk/actions

@ThePedroo ThePedroo added bug Something isn't working confirmed This issue or pull request is confirmed to be done. labels Nov 24, 2024
@ThePedroo ThePedroo merged commit d498a9a into PerformanC:main Nov 24, 2024
1 check passed
Trijal08 added a commit to Trijal08/ReZygisk that referenced this pull request Nov 25, 2024
fix: trigger Zygisk modules service.sh in parallel (PerformanC#91)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working confirmed This issue or pull request is confirmed to be done.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants