Skip to content
This repository has been archived by the owner on Dec 25, 2023. It is now read-only.

Update MSBuild.StructuredLogger to the latest version #249

Merged

Conversation

KirillOsenkov
Copy link
Contributor

This enables reading the latest binlogs produced by recent versions of MSBuild (including the latest version 14).

@KirillOsenkov KirillOsenkov requested a review from a team as a code owner September 1, 2021 17:27
@pascalberger
Copy link
Member

@KirillOsenkov Thanks for the PR 👍

We kept dependency at 2.0 by design for the moment, to maintain compatibility with all .NET versions Cake supports. Version 2.1 drops support for .NET 4.6 and MsBuild 14 and bumps requirements to .NET 4.7.2 and MsBuild to 15.1.0 which would mean that, while it would support newer binlog formats, it will stop working on these environments.
Note that binlog is only one of the supported formats of this addin (the other is an XML format created by MsBuild.Extension.Pack logger).

I personally can live with this restriction, even if it means that requirements for the addin are higher than what Cake itself requires and removes support for older Visual Studio versions. So maybe we should just bump it, as I haven't heard any different opinion yet.

See #178 for details.

@KirillOsenkov
Copy link
Contributor Author

I see, thanks for the explanation. Indeed, bringing back MSBuild 14 and .NET 4.6 would require adding a lot of #ifdef and other plumbing, so doubtful it's worth doing. Then your current approach is sensible.

Copy link
Member

@pascalberger pascalberger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cake 2.0 has removed support for .NET Framework runner and therefore we can update to latest MsBuild.StructuredLogger in this addin

This enables reading the latest binlogs produced by recent versions of MSBuild (including the latest version 14).
@pascalberger pascalberger linked an issue Dec 4, 2021 that may be closed by this pull request
@pascalberger pascalberger merged commit 1a33793 into cake-contrib:develop Dec 4, 2021
@pascalberger
Copy link
Member

@KirillOsenkov your changes have been merged, thanks for your contribution 👍

@KirillOsenkov KirillOsenkov deleted the dev/kirillo/updateBinlog branch December 4, 2021 21:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for binlog version 9 file format
2 participants