Skip to content

Comments

Refactor LSPosed integration using composite builds#71

Merged
JingMatrix merged 3 commits intomasterfrom
vector
Feb 13, 2026
Merged

Refactor LSPosed integration using composite builds#71
JingMatrix merged 3 commits intomasterfrom
vector

Conversation

@JingMatrix
Copy link
Owner

We change the integration with the core repository (JingMatrix/LSPosed) from manual subproject inclusion to a Gradle composite build.

Key changes include:

  • Replaced direct include and projectDir mappings in settings.gradle.kts with includeBuild("core").
  • Configured dependency substitutions to map 'vector' module coordinates to the appropriate projects within the included build.
  • Updated dependency declarations across all subprojects to use the substituted coordinates.
  • Removed share/lspatch-rules.pro and the corresponding afterEvaluate logic in the root build script, as local testing confirms these additional ProGuard rules are no longer required.

We change the integration with the core repository (JingMatrix/LSPosed) from manual subproject inclusion to a Gradle composite build.

Key changes include:
- Replaced direct `include` and `projectDir` mappings in `settings.gradle.kts` with `includeBuild("core")`.
- Configured dependency substitutions to map 'vector' module coordinates to the appropriate projects within the included build.
- Updated dependency declarations across all subprojects to use the substituted coordinates.
- Removed `share/lspatch-rules.pro` and the corresponding `afterEvaluate` logic in the root build script, as local testing confirms these additional ProGuard rules are no longer required.
Switch from eager resolution of `runtimeClasspath` to the Provider API's lazy `map` implementation in the jar configuration.

This change resolves a `beforeEvaluate` lifecycle error encountered during composite builds, as it prevents the early evaluation of included builds during the configuration phase.
@JingMatrix JingMatrix merged commit cf361b2 into master Feb 13, 2026
1 check passed
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