Skip to content

Commit ebe411a

Browse files
committed
Update dependencies (new utils updates)
1 parent a9342c3 commit ebe411a

File tree

4 files changed

+33
-29
lines changed

4 files changed

+33
-29
lines changed

settings.gradle

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,10 @@ dependencyResolutionManagement.versionCatalogs.register('libs') {
2828
library 'gson', 'com.google.code.gson', 'gson' version '2.11.0'
2929
library 'jopt', 'net.sf.jopt-simple', 'jopt-simple' version '6.0-alpha-3'
3030

31-
library 'utils-download', 'net.minecraftforge', 'download-utils' version '0.3.0'
32-
library 'utils-hash', 'net.minecraftforge', 'hash-utils' version '0.1.9'
33-
library 'utils-data', 'net.minecraftforge', 'json-data-utils' version '0.2.1'
34-
library 'utils-logging', 'net.minecraftforge', 'log-utils' version '0.3.2'
31+
library 'utils-download', 'net.minecraftforge', 'download-utils' version '0.4.0'
32+
library 'utils-hash', 'net.minecraftforge', 'hash-utils' version '0.1.12'
33+
library 'utils-data', 'net.minecraftforge', 'json-data-utils' version '0.4.0'
34+
library 'utils-logging', 'net.minecraftforge', 'log-utils' version '0.5.0'
3535
bundle 'utils', [ 'utils-download', 'utils-hash', 'utils-data', 'utils-logging' ]
3636

3737
library 'nulls', 'org.jetbrains', 'annotations' version '26.0.2'

src/main/java/net/minecraftforge/launcher/Constants.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,13 @@
55
package net.minecraftforge.launcher;
66

77
import net.minecraftforge.util.data.MCJsonUtils;
8+
import net.minecraftforge.util.os.OS;
89

910
import java.io.File;
1011

11-
interface Constants {
12-
File ASSETS_DIR = new File(MCJsonUtils.getMCDir(), "assets");
13-
String RESOURCES_URL = "https://resources.download.minecraft.net/";
12+
final class Constants {
13+
private Constants() { }
14+
15+
static final File ASSETS_DIR = new File(MCJsonUtils.getMCDir(OS.current()), "assets");
16+
static final String RESOURCES_URL = "https://resources.download.minecraft.net/";
1417
}

src/main/java/net/minecraftforge/launcher/DownloadAssets.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
import net.minecraftforge.util.data.json.MinecraftVersion;
1010
import net.minecraftforge.util.download.DownloadUtils;
1111
import net.minecraftforge.util.hash.HashFunction;
12-
import net.minecraftforge.util.logging.Log;
1312

1413
import java.io.File;
1514
import java.util.Map;
@@ -36,17 +35,17 @@ static void download(String repo, File assetsDir, MinecraftVersion versionJson)
3635
String assetDest = getAssetDest(asset.hash);
3736
File file = new File(objectsDir, assetDest);
3837
if (file.exists()) {
39-
Log.debug("Considering existing file with size " + file.length() + " for " + name);
38+
Main.LOGGER.debug("Considering existing file with size " + file.length() + " for " + name);
4039
if (file.length() == asset.size) {
41-
Log.debug("Size check succeeded. Skipping.");
40+
Main.LOGGER.debug("Size check succeeded. Skipping.");
4241
continue;
4342
}
4443
}
4544

4645
// We need to download assets? Release the log so the consumer is aware.
47-
Log.release();
46+
Main.LOGGER.release();
4847
try {
49-
Log.info("Downloading missing asset: " + name);
48+
Main.LOGGER.info("Downloading missing asset: " + name);
5049
DownloadUtils.downloadFile(file, repo + assetDest);
5150
String newSha1 = HashFunction.SHA1.sneakyHash(file);
5251
if (!newSha1.equals(asset.hash)) {

src/main/java/net/minecraftforge/launcher/Main.java

Lines changed: 19 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
import joptsimple.OptionSet;
1111
import net.minecraftforge.util.data.json.JsonData;
1212
import net.minecraftforge.util.data.json.MinecraftVersion;
13-
import net.minecraftforge.util.logging.Log;
13+
import net.minecraftforge.util.logging.Logger;
1414

1515
import java.io.File;
1616
import java.io.FileNotFoundException;
@@ -24,25 +24,27 @@
2424
import java.util.zip.ZipFile;
2525

2626
public final class Main {
27+
static final Logger LOGGER = Logger.create();
28+
2729
public static void main(String[] args) throws Throwable {
2830
long start = System.currentTimeMillis();
2931
Launcher launcher;
3032
try {
31-
Log.capture();
33+
LOGGER.capture();
3234
launcher = run(args);
3335
} catch (Throwable e) {
34-
Log.release();
36+
LOGGER.release();
3537
throw e;
3638
}
3739

3840
long total = System.currentTimeMillis() - start;
39-
if (Log.isCapturing()) {
40-
Log.drop();
41-
Log.INFO.print("Slime Launcher setup is up-to-date");
41+
if (LOGGER.isCapturing()) {
42+
LOGGER.drop();
43+
LOGGER.getInfo().print("Slime Launcher setup is up-to-date");
4244
} else {
43-
Log.INFO.print("Slime Launcher has finished setting up");
45+
LOGGER.getInfo().print("Slime Launcher has finished setting up");
4446
}
45-
Log.INFO.println(", took " + total + "ms\n");
47+
LOGGER.getInfo().println(", took " + total + "ms\n");
4648

4749
launcher.run();
4850
}
@@ -82,14 +84,14 @@ private static Launcher run(String[] args) throws Throwable {
8284
.withRequiredArg().ofType(String.class);
8385

8486
Package pkg = Main.class.getPackage();
85-
Log.info(pkg.getImplementationTitle() + " " + pkg.getImplementationVersion());
87+
LOGGER.info(pkg.getImplementationTitle() + " " + pkg.getImplementationVersion());
8688

8789
SplitArgs split = new SplitArgs(args);
8890

8991
OptionSet options = parser.parse(split.sl);
9092
if (options.has(helpO)) {
91-
parser.printHelpOn(Log.INFO);
92-
Log.info("To pass arguments into the main class,\n" +
93+
parser.printHelpOn(LOGGER.getInfo());
94+
LOGGER.info("To pass arguments into the main class,\n" +
9395
"add '--' after the Slime Launcher arguments,\n" +
9496
"followed by your main arguments.");
9597
throw new IllegalArgumentException("Incomplete or invalid arguments");
@@ -109,12 +111,12 @@ private static Launcher run(String[] args) throws Throwable {
109111
);
110112
}
111113

112-
Log.info("Checking assets");
113-
byte indent = Log.push();
114+
LOGGER.info("Checking assets");
115+
byte indent = LOGGER.push();
114116
DownloadAssets.download(assetsRepo, assets, versionJson);
115-
Log.pop(indent);
117+
LOGGER.pop(indent);
116118

117-
Log.info("Looking for main class: " + mainClass);
119+
LOGGER.info("Looking for main class: " + mainClass);
118120
MethodHandle mainMethod;
119121
try {
120122
Class<?> main = Class.forName(mainClass);
@@ -123,7 +125,7 @@ private static Launcher run(String[] args) throws Throwable {
123125
throw new IllegalStateException("Could not find main class!", e);
124126
}
125127

126-
Log.info("Sanitizing Minecraft arguments");
128+
LOGGER.info("Sanitizing Minecraft arguments");
127129
for (int i = 0; i < split.mc.length; i++) {
128130
split.mc[i] = split.mc[i]
129131
.replace("{asset_index}", versionJson.assetIndex.id)
@@ -146,7 +148,7 @@ private Launcher(String name, MethodHandle main, String[] args) {
146148

147149
@SuppressWarnings("ConfusingArgumentToVarargsMethod")
148150
private void run() throws Throwable {
149-
Log.info("Launching using main class: " + this.name);
151+
LOGGER.info("Launching using main class: " + this.name);
150152
this.main.invokeExact(this.args);
151153
}
152154
}

0 commit comments

Comments
 (0)