Age | Commit message (Collapse) | Author |
|
|
|
We've assumed that Forge adds its annotation everywhere via its patches but it
actually only adds them if a given file needs to have a patch applied for
reasons other than just the SideOnly annotation.
This would leave some classes/fields/methods without annotation which resulted
in an exception when deserializing horses on 1.12.2 (because it references a
class which in turn has methods referencing the dedicated server class which is
server-only).
Instead, we now convert all Fabric annotations (added during merging) to the
Forge ones and remove duplicates (where patches also added them).
|
|
|
|
|
|
I do not quite remember why I added this in the first place but it seems fine
without, so let's not mess with it.
|
|
|
|
|
|
|
|
Usages are more related to the modern Forge than to ModLauncher specifically.
|
|
|
|
|
|
Cause modern versions of Forge do not need them.
|
|
We never use the v1 file, so there isn't really any reason to keep it around.
|
|
|
|
A fallback SrgProvider is instantiated and initialized here but never added to
the provider list, resulting in a NPE right below in `getRawSrgFile`.
|
|
This jar used to always be included in the cache file list even though it is
only used/generated with official mappings. Therefore it would always be missing
from the cache when not using official mappings, unnecessarily re-running the
patching code.
|
|
Way easier to read than the previous array-copy implementation and trivial to
extend when more file are optional (see next commit).
|
|
|
|
Signed-off-by: shedaniel <daniel@shedaniel.me>
|
|
|
|
|
|
|
|
Signed-off-by: shedaniel <daniel@shedaniel.me>
|
|
Signed-off-by: shedaniel <daniel@shedaniel.me>
# Conflicts:
# build.gradle
# src/main/java/net/fabricmc/loom/configuration/mods/ModProcessor.java
|
|
Signed-off-by: shedaniel <daniel@shedaniel.me>
|
|
Signed-off-by: shedaniel <daniel@shedaniel.me>
|
|
* Juuzify ArchitecturyLoomDecompiler
* Remove unused max memory
|
|
* Add architectury decompiler
Signed-off-by: shedaniel <daniel@shedaniel.me>
* Perhaps let's not include javaexec helpers
Signed-off-by: shedaniel <daniel@shedaniel.me>
* Fix checkstyle and resolve reviews
Signed-off-by: shedaniel <daniel@shedaniel.me>
|
|
* Validate remapped mods also have a remapped access widener.
* Fix checkstyle
|
|
Signed-off-by: shedaniel <daniel@shedaniel.me>
|
|
Signed-off-by: shedaniel <daniel@shedaniel.me>
|
|
Signed-off-by: shedaniel <daniel@shedaniel.me>
|
|
# Conflicts:
# src/main/java/net/fabricmc/loom/configuration/providers/mappings/LayeredMappingSpecBuilderImpl.java
# src/main/java/net/fabricmc/loom/configuration/providers/mappings/mojmap/MojangMappingLayer.java
# src/main/java/net/fabricmc/loom/configuration/providers/mappings/mojmap/MojangMappingsSpec.java
# src/test/groovy/net/fabricmc/loom/test/unit/layeredmappings/LayeredMappingSpecBuilderTest.groovy
|
|
Unpick will now get the same version of asm that loom is running with on its classpath.
|
|
A more sophisticated fix might be nice, but im not sure it will provide a lot of benefit.
|
|
|
|
|
|
from the official mojang mappings. (#538)
|
|
# Conflicts:
# src/main/java/net/fabricmc/loom/configuration/providers/minecraft/MinecraftMappedProvider.java
|
|
When using transitive access wideners the mc jar would always be re-processed.
|
|
|
|
Signed-off-by: shedaniel <daniel@shedaniel.me>
|
|
# Conflicts:
# src/main/java/net/fabricmc/loom/configuration/providers/mappings/MappingsProviderImpl.java
# src/test/groovy/net/fabricmc/loom/test/integration/FabricAPITest.groovy
|
|
* Fix `namedElements` not extending from api
Fix artifact not waiting for remapAllJars with shared caches
* Use apiElements
* Add a better test for this.
* Also add remapped api mods to namedElements
|
|
In the bundler its called 1.18 Pre-release 1 not 1.18-pre1, so we just assume there is only ever going to be 1 version per jar.
|
|
This will be used on a lot more projects now as transitive access wideners becomes more widely used.
|
|
|
|
|
|
# Conflicts:
# src/main/java/net/fabricmc/loom/api/LoomGradleExtensionAPI.java
# src/main/java/net/fabricmc/loom/extension/MinecraftGradleExtension.java
|
|
|