Skip to content

Support remapping Jar Manifest entries - #23

Merged
Technici4n merged 2 commits into
mainfrom
remap-manifest
Nov 2, 2025
Merged

Support remapping Jar Manifest entries#23
Technici4n merged 2 commits into
mainfrom
remap-manifest

Conversation

@shartte

@shartte shartte commented Nov 2, 2025

Copy link
Copy Markdown
Contributor

Add support for remapping entries (the per-file attributes) in the Jar Manifest, which is required to support the source-distribution entries we now add to merged jars to track which source distribution each file came from.

@neoforged-pr-publishing

neoforged-pr-publishing Bot commented Nov 2, 2025

Copy link
Copy Markdown
  • Publish PR to GitHub Packages

Last commit published: 4d0e34b330d04e199175300e01ae68551ef23936 - version: 2.0.18-pr-23-remap-manifest

PR Publishing

The artifacts published by this PR:

Repository Declaration

In order to use the artifacts published by the PR, add the following repository to your buildscript:

repositories {
    maven {
        name = "Maven for PR #23" // https://github.com/neoforged/AutoRenamingTool/pull/23
        url = uri("https://prmaven.neoforged.net/AutoRenamingTool/pr23")
        content {
            includeModule("net.neoforged", "autorenamingtool")
        }
    }
}

@shartte
shartte marked this pull request as draft November 2, 2025 14:52
@shartte shartte changed the title Support devtime merging. Support remapping Jar Manifest entries Nov 2, 2025

@Technici4n Technici4n left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, just one remark maybe

Comment thread src/main/java/net/neoforged/art/internal/EntryImpl.java
@shartte
shartte marked this pull request as ready for review November 2, 2025 16:01
@Technici4n
Technici4n merged commit d509ba3 into main Nov 2, 2025
3 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