Skip to content

Commit

Permalink
[C#] fix dotnetastgen runner to handle space containing paths (#4419)
Browse files Browse the repository at this point in the history
  • Loading branch information
karan-batavia authored Apr 2, 2024
1 parent 93567c7 commit 1a2b5c9
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion joern-cli/frontends/csharpsrc2cpg/build.sbt
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ enablePlugins(JavaAppPackaging, LauncherJarPlugin)

lazy val AstgenWin = "dotnetastgen-win.exe"
lazy val AstgenLinux = "dotnetastgen-linux"
lazy val AstgenLinuxArm = "dotnetastgen-linux-arm"
lazy val AstgenLinuxArm = "dotnetastgen-linux-arm64"
lazy val AstgenMac = "dotnetastgen-macos"

lazy val AllPlatforms = Seq(AstgenWin, AstgenLinux, AstgenLinuxArm, AstgenMac)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
csharpsrc2cpg {
dotnetastgen_version: "0.29.0"
dotnetastgen_version: "0.34.0"
}
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ class DotNetAstGenRunner(config: Config) extends AstGenRunnerBase(config) {
metaData: AstGenProgramMetaData
): Try[Seq[String]] = {
val excludeCommand = if (exclude.isEmpty) "" else s"-e \"$exclude\""
ExternalCommand.run(s"$astGenCommand -o ${out.toString()} -i $in $excludeCommand", ".")
ExternalCommand.run(s"$astGenCommand -o ${out.toString()} -i \"$in\" $excludeCommand", ".")
}

}

0 comments on commit 1a2b5c9

Please sign in to comment.