Skip to content

Update gradle buildscripts in README.md#221

Merged
hyperxpro merged 3 commits into
hyperxpro:mainfrom
solonovamax:main
Jul 22, 2025
Merged

Update gradle buildscripts in README.md#221
hyperxpro merged 3 commits into
hyperxpro:mainfrom
solonovamax:main

Conversation

@solonovamax

Copy link
Copy Markdown
Contributor

Motivation:

Gradle buildscript was broken and it also kinda sucked.

Modification:

Changes:

  • Fixes gradle buildscripts when used with the latest version (8.14.3)
  • Replaces dynamic dependency with a custom dependency metadata rule
  • Includes support for shading all natives into a single jar using shadow

Result:
Resolves #219
Fixes #142
Closes #218

solonovamax and others added 2 commits July 16, 2025 15:12
Changes:
- Fixes gradle buildscripts when used with the latest version (8.14.3)
- Replaces dynamic dependency with a custom dependency metadata rule
- Includes support for shading all natives into a single jar using shadow

Signed-off-by: solonovamax <solonovamax@12oclockpoint.com>
@hyperxpro

Copy link
Copy Markdown
Owner

I wonder if we can make Gradle an expandable section since it is quite large now.

@solonovamax

Copy link
Copy Markdown
Contributor Author

I wonder if we can make Gradle an expandable section since it is quite large now.

sure I could do that if you'd like

@hyperxpro

Copy link
Copy Markdown
Owner

I wonder if we can make Gradle an expandable section since it is quite large now.

sure I could do that if you'd like

please do it :)

Changes:
- Put large portion of gradle section into a <details> block.
- Add alerts for some important information
- Properly align Windows 11 row in supported platforms table

Signed-off-by: solonovamax <solonovamax@12oclockpoint.com>
@solonovamax

Copy link
Copy Markdown
Contributor Author

please do it :)

@hyperxpro done

I've also added the following other changes:

  • move some important information into alerts:
    • the instruction to "Install Microsoft Visual C++ Redistributable before running this library on Windows" in an "important" alert
    • the note for "creating a distribution zip/tar" with gradle into a "note" alert
    • added an important alert to the gradle section to not forget to add the metadata rule
    • made the portion saying to call Brotli4jLoader.ensureAvailability() into a warning alert
    • made the portion saying it's possible to load the native library manually from a custom path into a tip alert
    • made the note about Windows-AArch64 into a note alert
    • made the note about RISC-V64 into an important alert
  • re-align the supported platforms table (this change is not reflected visually, it's exclusively relevant to someone viewing the readme in any form of text editor)

it doesn't render the best in the github diff, so you can see it properly rendered on my fork

@hyperxpro hyperxpro merged commit a43b2a8 into hyperxpro:main Jul 22, 2025
28 checks passed
@hyperxpro

Copy link
Copy Markdown
Owner

@solonovamax Thanks a lot for the great work!

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.

Improved gradle buildscript Groovy scripts in ReadMe not works

2 participants