Cleanup runtime only log4j, allow it on any version.
This commit is contained in:
		
							parent
							
								
									e180e39575
								
							
						
					
					
						commit
						2c5baf95cb
					
				
					 2 changed files with 2 additions and 6 deletions
				
			
		|  | @ -42,7 +42,7 @@ public class MinecraftLibraryProvider { | ||||||
| 		final MinecraftJarConfiguration jarConfiguration = extension.getMinecraftJarConfiguration().get(); | 		final MinecraftJarConfiguration jarConfiguration = extension.getMinecraftJarConfiguration().get(); | ||||||
| 		final MinecraftVersionMeta versionInfo = minecraftProvider.getVersionInfo(); | 		final MinecraftVersionMeta versionInfo = minecraftProvider.getVersionInfo(); | ||||||
| 		final BundleMetadata serverBundleMetadata = minecraftProvider.getServerBundleMetadata(); | 		final BundleMetadata serverBundleMetadata = minecraftProvider.getServerBundleMetadata(); | ||||||
| 		final boolean runtimeOnlyLog4j = versionInfo.isVersionOrNewer(Constants.MinecraftReleaseTimes.MC_22W03A) && extension.getRuntimeOnlyLog4j().get(); | 		final boolean runtimeOnlyLog4j = extension.getRuntimeOnlyLog4j().get(); | ||||||
| 
 | 
 | ||||||
| 		final boolean overrideLWJGL = LWJGLVersionOverride.overrideByDefault() || LWJGLVersionOverride.forceOverride(project) || Boolean.getBoolean("loom.test.lwjgloverride"); | 		final boolean overrideLWJGL = LWJGLVersionOverride.overrideByDefault() || LWJGLVersionOverride.forceOverride(project) || Boolean.getBoolean("loom.test.lwjgloverride"); | ||||||
| 
 | 
 | ||||||
|  | @ -57,7 +57,7 @@ public class MinecraftLibraryProvider { | ||||||
| 
 | 
 | ||||||
| 			if (library.isValidForOS() && !library.hasNatives() && library.artifact() != null) { | 			if (library.isValidForOS() && !library.hasNatives() && library.artifact() != null) { | ||||||
| 				if (runtimeOnlyLog4j && library.name().startsWith("org.apache.logging.log4j")) { | 				if (runtimeOnlyLog4j && library.name().startsWith("org.apache.logging.log4j")) { | ||||||
| 					// Make log4j a runtime only dep in 20w03a or later. Modders should use SLF4J. | 					// Make log4j a runtime only dep to force slf4j. | ||||||
| 					project.getDependencies().add(Constants.Configurations.MINECRAFT_RUNTIME_DEPENDENCIES, library.name()); | 					project.getDependencies().add(Constants.Configurations.MINECRAFT_RUNTIME_DEPENDENCIES, library.name()); | ||||||
| 				} else if (serverBundleMetadata != null && isLibraryInBundle(serverBundleMetadata, library)) { | 				} else if (serverBundleMetadata != null && isLibraryInBundle(serverBundleMetadata, library)) { | ||||||
| 					project.getDependencies().add(Constants.Configurations.MINECRAFT_SERVER_DEPENDENCIES, library.name()); | 					project.getDependencies().add(Constants.Configurations.MINECRAFT_SERVER_DEPENDENCIES, library.name()); | ||||||
|  |  | ||||||
|  | @ -142,8 +142,4 @@ public class Constants { | ||||||
| 		private TaskGroup() { | 		private TaskGroup() { | ||||||
| 		} | 		} | ||||||
| 	} | 	} | ||||||
| 
 |  | ||||||
| 	public static final class MinecraftReleaseTimes { |  | ||||||
| 		public static final String MC_22W03A = "2022-01-19T16:04:59+00:00"; |  | ||||||
| 	} |  | ||||||
| } | } | ||||||
|  |  | ||||||
		Loading…
	
		Reference in a new issue