-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replay #1793
Replay #1793
Conversation
Instead of specifying a project or solution, you can specify a .binlog file to "build" and it will replay all the events in the source log into the usual loggers.
@dotnet-bot test Windows_NT Build for Desktop please |
@dotnet-bot test Ubuntu16.04 Build for CoreCLR |
{ | ||
string code; | ||
string helpKeyword; | ||
var text = ResourceUtilities.FormatResourceString(out code, out helpKeyword, "InvalidLogFileFormat", ex.Message); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there appears to be a FormatResourceString overload that does not have the out params
ReplayBinaryLog(projectFile, loggers); | ||
} | ||
else // regular build | ||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: The new else block is not indented.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, I like how cleanly everything fit in. Will you also write some unit tests in a future PR? :D
There's already a test for replay here: |
Binary log replay functionality.
Instead of specifying a project or solution, you can specify a .binlog file to "build" and it will replay all the events in the source log into the usual loggers.
Note that BuildLogReplayEventArgs and its base class EventArgsDispatcher need to be made public for the replay functionality in MSBuild.exe to work. I've made the class sealed and added with explanations.