Skip to content

Commit

Permalink
Merge remote-tracking branch 'upstream/master'
Browse files Browse the repository at this point in the history
# Conflicts:
#	src/main/java/net/runelite/launcher/Launcher.java
  • Loading branch information
Therealnull committed Jul 22, 2019
2 parents f463474 + 9ae9444 commit 5574822
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 4 deletions.
7 changes: 6 additions & 1 deletion src/main/java/net/runelite/launcher/Launcher.java
Expand Up @@ -81,6 +81,7 @@ public class Launcher
private static final String USER_AGENT = "RuneLite/" + PROPERTIES.getVersion();
private static final boolean enforceDependencyHashing = true;
private static final boolean staging = false;
private static final File CRASH_FILES = new File(LOGS_DIR, "jvm_crash_pid_%p.log");

static final String CLIENT_MAIN_CLASS = "net.runelite.client.RuneLite";

Expand Down Expand Up @@ -150,6 +151,10 @@ public static void main(String[] args)
// Set all JVM params
setJvmParams(extraJvmParams);

// Set hs_err_pid location (do this after setJvmParams because it can't be set at runtime)
log.debug("Setting JVM crash log location to {}", CRASH_FILES);
extraJvmParams.add("-XX:ErrorFile=" + CRASH_FILES.getAbsolutePath());

try
{
UIManager.setLookAndFeel(UIManager.getCrossPlatformLookAndFeelClassName());
Expand All @@ -176,7 +181,7 @@ public static void main(String[] args)
}

// update packr vmargs
PackrConfig.updateLauncherArgs(bootstrap);
PackrConfig.updateLauncherArgs(bootstrap, extraJvmParams);

REPO_DIR.mkdirs();

Expand Down
13 changes: 10 additions & 3 deletions src/main/java/net/runelite/launcher/PackrConfig.java
Expand Up @@ -32,6 +32,9 @@
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.PrintWriter;
import java.util.Arrays;
import java.util.Collection;
import java.util.List;
import java.util.Map;
import lombok.extern.slf4j.Slf4j;
import net.runelite.launcher.beans.Bootstrap;
Expand All @@ -40,7 +43,7 @@
public class PackrConfig
{
// Update the packr vmargs
public static void updateLauncherArgs(Bootstrap bootstrap)
public static void updateLauncherArgs(Bootstrap bootstrap, Collection<String> extraJvmArgs)
{
File configFile = new File("config.json").getAbsoluteFile();

Expand All @@ -63,13 +66,17 @@ public static void updateLauncherArgs(Bootstrap bootstrap)
return;
}

String[] args = getArgs(bootstrap);
if (args == null || args.length == 0)
String[] argsArr = getArgs(bootstrap);
if (argsArr == null || argsArr.length == 0)
{
log.warn("Launcher args are empty");
return;
}

// Insert JVM arguments to config.json because some of them require restart
List<String> args = Arrays.asList(argsArr);
args.addAll(extraJvmArgs);

config.put("vmArgs", args);

try (PrintWriter writer = new PrintWriter(new FileOutputStream(configFile)))
Expand Down

0 comments on commit 5574822

Please sign in to comment.