Skip to content

移除 Pack200 支持#4267

Merged
Glavo merged 1 commit intoHMCL-dev:mainfrom
Glavo:remove-pack200
Aug 16, 2025
Merged

移除 Pack200 支持#4267
Glavo merged 1 commit intoHMCL-dev:mainfrom
Glavo:remove-pack200

Conversation

@Glavo
Copy link
Member

@Glavo Glavo commented Aug 15, 2025

目前 HMCL 不再兼容 Java 8,用户所用的 Java 支持 Pack200 的可能性大大降低,继续保留对 Pack200 的支持缺乏意义,所以让我们彻底移除这部分代码。

@Glavo Glavo requested a review from Copilot August 15, 2025 07:34
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 PR removes Pack200 support from HMCL as the application no longer supports Java 8, making Pack200 compatibility unnecessary since newer Java versions have deprecated or removed Pack200 functionality.

  • Completely removes the Pack200Utils utility class
  • Updates RemoteVersion to remove Pack200-related download logic and types
  • Removes Pack200 decompression handling from HMCLDownloadTask
  • 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 Removes Pack200 download option and PACK_XZ type enum
HMCL/src/main/java/org/jackhuang/hmcl/upgrade/HMCLDownloadTask.java Removes Pack200 decompression logic and XZ dependencies
HMCL/build.gradle.kts Removes pack200.jar from classpath configuration

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

@Glavo Glavo merged commit a4f9438 into HMCL-dev:main Aug 16, 2025
2 checks passed
@Glavo Glavo deleted the remove-pack200 branch August 16, 2025 07:45
Glavo added a commit to Glavo/HMCL that referenced this pull request Oct 13, 2025
Glavo added a commit that referenced this pull request Oct 13, 2025
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