Skip to content

[release/3.6] 移除 Pack200 支持#4666

Merged
Glavo merged 1 commit intoHMCL-dev:release/3.6from
Glavo:backport/3.6/4267
Oct 13, 2025
Merged

[release/3.6] 移除 Pack200 支持#4666
Glavo merged 1 commit intoHMCL-dev:release/3.6from
Glavo:backport/3.6/4267

Conversation

@Glavo
Copy link
Member

@Glavo Glavo commented Oct 13, 2025

@Glavo Glavo added the backport label Oct 13, 2025
@Glavo Glavo changed the title 移除 Pack200 支持 [release/3.6] 移除 Pack200 支持 Oct 13, 2025
@Glavo Glavo requested a review from Copilot October 13, 2025 07:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request removes Pack200 support from the HMCL project. Pack200 was a Java compression format that has been deprecated and removed from modern Java versions.

  • Removes the entire Pack200Utils utility class
  • Updates RemoteVersion to only support JAR downloads, removing Pack200/XZ support
  • Simplifies HMCLDownloadTask by removing Pack200 decompression logic
  • Removes Pack200-related classpath entry from build configuration

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
HMCLCore/src/main/java/org/jackhuang/hmcl/util/Pack200Utils.java Complete removal of Pack200 utility class
HMCL/src/main/java/org/jackhuang/hmcl/upgrade/RemoteVersion.java Simplified to only support JAR downloads, removed Pack200 logic
HMCL/src/main/java/org/jackhuang/hmcl/upgrade/HMCLDownloadTask.java Removed Pack200 decompression handling
HMCL/build.gradle.kts Removed Pack200 classpath entry from manifest

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Glavo Glavo merged commit b5d3653 into HMCL-dev:release/3.6 Oct 13, 2025
2 checks passed
@Glavo Glavo deleted the backport/3.6/4267 branch October 13, 2025 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants