Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Do not minimize the shadow jar #1285

Merged
merged 2 commits into from
Jul 11, 2022
Merged

Do not minimize the shadow jar #1285

merged 2 commits into from
Jul 11, 2022

Conversation

cmnrd
Copy link
Collaborator

@cmnrd cmnrd commented Jul 7, 2022

Issue #1284 was caused by the mimization step of the shadowJar gradle plugin. The main problem is, that we don't import kotlin classes directly, but load them manually in order to work around the build problems in Eclipse... Hence, the minimizer thinks it can exclude the Kotlin classes. While in principle it is possible to add particular excludes, this does not work properly when we compile a second time. The reason for this is unclear, but we might as well disable the minimization. This slightly increases the jar size from 23 MB to 28 MB.

Closes #1284

Issue #1284 was caused by the mimization step of the shadowJar gradle plugin.
The main problem is, that we don't import kotlin classes directly, but load
them manually in order to work around the build problems in Eclipse... Hence,
the minimizer thinks it can exclude the Kotlin classes. While in
principle it is possible to add particular excludes, this does not work properly
when we compile a second time. The reason for this is unclear, but we might as
well disable the minimization. This slightly increases the jar size from 23 MB to
28 MB.

Closes #1284
@cmnrd cmnrd requested review from lhstrh and petervdonovan July 7, 2022 14:33
@cmnrd cmnrd added bug Something isn't working gradle Issues regarding Gradle build configuration labels Jul 8, 2022
Copy link
Member

@lhstrh lhstrh left a comment

Choose a reason for hiding this comment

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

I think this is a fine solution, but we should leave a note in the Gradle config explaining why we aren't doing minimization. We could just refer to this PR.

@cmnrd cmnrd merged commit 12f8ec1 into master Jul 11, 2022
@cmnrd cmnrd deleted the fix-1284 branch July 11, 2022 07:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working gradle Issues regarding Gradle build configuration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gradle does not include Kotlin classes when compiling again
2 participants