-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Fix named attributes in legacy failing when reading back data from Module Metadata #225
Conversation
Last commit published: d7affafcb0aaa5da1cc75508ad489241677e3d82. PR PublishingThe artifacts published by this PR:
Repository DeclarationIn order to use the artifacts published by the PR, add the following repository to your buildscript: repositories {
maven {
name 'Maven for PR #225' // https://github.com/neoforged/ModDevGradle/pull/225
url 'https://prmaven.neoforged.net/ModDevGradle/pr225'
content {
includeModule('net.neoforged.moddev', 'net.neoforged.moddev.gradle.plugin')
includeModule('net.neoforged', 'moddev-gradle')
includeModule('net.neoforged.moddev.repositories', 'net.neoforged.moddev.repositories.gradle.plugin')
includeModule('net.neoforged.moddev.legacyforge', 'net.neoforged.moddev.legacyforge.gradle.plugin')
}
}
} |
The fix is working, I just checked it myself. Thanks |
The fix, as it turned out, partially works. The dependency is added, but the reobf call does not occur, thus the game starts with the SRG version of the mod, and not its reobf version. |
@Icosider Can you try again with the latest version of this PR? |
@shartte, It seems to be working pretty well. But at the moment, publishing to maven using multiple reobfuscate(task, sourceSet) remains problematic(perhaps it will relate to another PR) |
Yes the publishing side is another story and unaffected by this. |
@Matyrobbrt Can you review? |
🚀 This PR has been released as ModDevGradle version |
We current have a bug where depending on a Mod that was published by MDG Legacy fails, due to that Mod containing our mappings attribute with the value "srg".
Gradle seems to be unable to map that back for some reason.