Merge pull request #75 from patchkit-net/bugfix/v3.10.0/1004-wrapping-patcher-data-location-in-qutoes

genail · web-flow · commit 2188406f3383 · 2018-07-24T14:08:39.000+02:00
Bugfix/v3.10.0/1004 wrapping patcher data location in qutoes
diff --git a/Assets/PatchKit Patcher/Scripts/AppStarter.cs b/Assets/PatchKit Patcher/Scripts/AppStarter.cs
@@ -72,7 +72,7 @@ private ProcessStartInfo GetProcessStartInfo(string executablePath, PlatformType
                     return new ProcessStartInfo
                     {
                         FileName = executablePath,
-                        Arguments = string.Format("+patcher-data-location {0}", _app.LocalMetaData.GetFilePath()),
+                        Arguments = string.Format("+patcher-data-location \"{0}\"", _app.LocalMetaData.GetFilePath()),
                         WorkingDirectory = workingDir
                     };
                 case PlatformType.OSX:
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -36,6 +36,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Fixed
 - Handling of the ZLib exception
+- Invalid handling of patcher-data-location argument with spaces
 
 ## [3.9.2]
 ### Added

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ private ProcessStartInfo GetProcessStartInfo(string executablePath, PlatformType`
`72`	`72`	`return new ProcessStartInfo`
`73`	`73`	`{`
`74`	`74`	`FileName = executablePath,`
`75`		`- Arguments = string.Format("+patcher-data-location {0}", _app.LocalMetaData.GetFilePath()),`
	`75`	`+ Arguments = string.Format("+patcher-data-location \"{0}\"", _app.LocalMetaData.GetFilePath()),`
`76`	`76`	`WorkingDirectory = workingDir`
`77`	`77`	`};`
`78`	`78`	`case PlatformType.OSX:`