Skip to content

Commit d5e2e61

Browse files
ilspycmd: route update-check messages to stderr
stdout is the data channel: --resource may be writing a binary payload to it. Update-check notices on stdout would tail-append to the binary stream and corrupt extracted files. Move them to stderr (npm/cargo/pip convention) so meta-output stays out of the data path.
1 parent 28fffb7 commit d5e2e61

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

ICSharpCode.ILSpyCmd/IlspyCmdProgram.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,8 @@ private async Task<int> OnExecuteAsync(CommandLineApplication app)
276276
var checkResult = await updateCheckTask;
277277
if (null != checkResult && checkResult.UpdateRecommendation)
278278
{
279-
Console.WriteLine("You are not using the latest version of the tool, please update.");
280-
Console.WriteLine($"Latest version is '{checkResult.LatestVersion}' (yours is '{checkResult.RunningVersion}')");
279+
app.Error.WriteLine("You are not using the latest version of the tool, please update.");
280+
app.Error.WriteLine($"Latest version is '{checkResult.LatestVersion}' (yours is '{checkResult.RunningVersion}')");
281281
}
282282
}
283283
}

0 commit comments

Comments
 (0)