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

Gradle build improvement #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

carlosame
Copy link
Contributor

This is an initial fix for the Gradle build (I ran it with Gradle 7.5.1). The build is not perfect but is a better base to work on, depending on the maintainer's technical decisions. (For example I'd personally get rid of the Proguard task)

If one runs gradle build -x test it completes just fine.

Remaining problems:

  1. The getPhysicalFontReturnsPhysicalFontForFontFamily test fails.
  2. The (now commented out) license plugin fails due to a number of missing headers. Once the problems are fixed, could be uncommented.
  3. The creation of Jar files still need a few adjustments (which I didn't do because of the interaction with the Proguard task).
  4. gradle cobertura coveralls fails with a number of errors, and the coveralls-gradle-plugin is not maintained anymore.

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