Slim down runfiles in publish_binary (#481)

purkhusid · web-flow · commit d39a5d876fad · 2025-03-22T16:20:20.000Z
## Why?
Because we only need the user provided runfiles to be in the actual runfiles object. The rest of the files are copied to the correct location in the output directory.
diff --git a/dotnet/private/rules/publish_binary/publish_binary.bzl b/dotnet/private/rules/publish_binary/publish_binary.bzl
@@ -205,7 +205,7 @@ def _publish_binary_impl(ctx):
         DefaultInfo(
             executable = apphost_shim,
             files = depset([apphost_shim, main_dll, runtimeconfig, depsjson] + outputs),
-            runfiles = ctx.runfiles(files = [apphost_shim, main_dll, runtimeconfig, depsjson] + outputs + runfiles),
+            runfiles = ctx.runfiles(files = runfiles),
         ),
     ]
 

Original file line number	Diff line number	Diff line change
`@@ -205,7 +205,7 @@ def _publish_binary_impl(ctx):`
`205`	`205`	`DefaultInfo(`
`206`	`206`	`executable = apphost_shim,`
`207`	`207`	`files = depset([apphost_shim, main_dll, runtimeconfig, depsjson] + outputs),`
`208`		`- runfiles = ctx.runfiles(files = [apphost_shim, main_dll, runtimeconfig, depsjson] + outputs + runfiles),`
	`208`	`+ runfiles = ctx.runfiles(files = runfiles),`
`209`	`209`	`),`
`210`	`210`	`]`
`211`	`211`