Skip to content

Fix native resource packaging exclusions#40

Merged
QiuYe-123 merged 1 commit into
mainfrom
build/fix-native-resource-packaging
May 3, 2026
Merged

Fix native resource packaging exclusions#40
QiuYe-123 merged 1 commit into
mainfrom
build/fix-native-resource-packaging

Conversation

@QiuYe-123
Copy link
Copy Markdown
Member

Description

Fix Gradle native resource packaging so bundled runtime dependencies and per-classifier license directories are preserved in published JARs.

This change narrows the exclusion rule to only replace files that are regenerated for the current classifier instead of excluding the entire classifier directory. It also allows non-binary files under native/ to be packaged, which keeps licenses/ directories intact.

Type of Change

  • New feature (type: feature)
  • Bug fix (type: bugfix)
  • Refactoring (type: refactor)
  • Build/CI (type: build)
  • Documentation (type: docs)
  • Tests (type: tests)

Native Changes

  • This PR changes native C++ code (requires ci:native label for full platform testing)
  • No native changes

Testing

Please describe the tests you ran to verify your changes.

./gradlew assemble -x test -x buildNative -x cmakeConfigure
./gradlew spotlessCheck -x buildNative -x cmakeConfigure

Checklist

  • My code follows the project's code style
  • I have added tests that prove my fix/feature works
  • ./gradlew spotlessCheck passes

@QiuYe-123 QiuYe-123 requested a review from a team as a code owner May 3, 2026 05:45
@QiuYe-123 QiuYe-123 self-assigned this May 3, 2026
@QiuYe-123 QiuYe-123 added type: bugfix Bug fix type: build Build system or dependency changes labels May 3, 2026
@github-actions github-actions Bot added the Tests: Failed One or more tests failed label May 3, 2026
@QiuYe-123 QiuYe-123 added the Release: Major - 0.X.0 Releases focused on Content, changes to gameplay; While maintaining mostly API stability. label May 3, 2026
@github-actions github-actions Bot added Tests: Passed All tests passed and removed Tests: Failed One or more tests failed labels May 3, 2026
@QiuYe-123 QiuYe-123 merged commit 952db3b into main May 3, 2026
26 of 37 checks passed
@QiuYe-123 QiuYe-123 deleted the build/fix-native-resource-packaging branch May 3, 2026 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release: Major - 0.X.0 Releases focused on Content, changes to gameplay; While maintaining mostly API stability. Tests: Passed All tests passed type: bugfix Bug fix type: build Build system or dependency changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant