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

Use NFRT to get more robust asset downloading #12

Merged
merged 1 commit into from
Aug 15, 2024
Merged

Conversation

shartte
Copy link
Contributor

@shartte shartte commented Aug 15, 2024

Replace home-brew asset downloading with NFRT to get more robust (retries, reuse of launcher assets, etc.) asset downloading.

NFRT produces the JSON file used by mcp Start.java directly

Also fixes:

  • Config file for ART was not written using UTF-8 causing issues when paths contained non-ASCII characters
  • Logging in ApplyPatches was broken due to a swapped argument

@neoforged-pr-publishing
Copy link

  • Publish PR to GitHub Packages

@shartte shartte merged commit 069d45c into 1.21.2-dev Aug 15, 2024
4 checks 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.

2 participants