Commit f62b5db

authored

Print log4j store directory only when operating in GUI mode (#9695)

This fixes CommandLineTest.testCommandLineVersion() Looks like the test was broken by the System.out.println statement in Base.java 824567d

1 parent b59c91e commit f62b5dbCopy full SHA for f62b5db

File tree

-3

lines changed

-3

lines changed

Lines changed: 6 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -207,9 +207,6 @@ public Base(String[] args) throws Exception {`
`207`	`207`	`BaseNoGui.getPlatform().init();`
`208`	`208`
`209`	`209`	`BaseNoGui.initPortableFolder();`
`210`		`- // This configure the logs root folder`
`211`		`- System.out.println("Set log4j store directory " + BaseNoGui.getSettingsFolder().getAbsolutePath());`
`212`		`- System.setProperty("log4j.dir", BaseNoGui.getSettingsFolder().getAbsolutePath());`
`213`	`210`
`214`	`211`	`// Look for a possible "--preferences-file" parameter and load preferences`
`215`	`212`	`BaseNoGui.initParameters(args);`
`@@ -218,6 +215,12 @@ public Base(String[] args) throws Exception {`
`218`	`215`	`parser.parseArgumentsPhase1();`
`219`	`216`	`commandLine = !parser.isGuiMode();`
`220`	`217`
	`218`	`+ // This configure the logs root folder`
	`219`	`+ if (parser.isGuiMode()) {`
	`220`	`+ System.out.println("Set log4j store directory " + BaseNoGui.getSettingsFolder().getAbsolutePath());`
	`221`	`+ }`
	`222`	`+ System.setProperty("log4j.dir", BaseNoGui.getSettingsFolder().getAbsolutePath());`
	`223`	`+`
`221`	`224`	`BaseNoGui.checkInstallationFolder();`
`222`	`225`
`223`	`226`	`// If no path is set, get the default sketchbook folder for this platform`

Comments

(0)