KirillOsenkov / MSBuildStructuredLog

A logger for MSBuild that records a structured representation of executed targets, tasks, property and item values.
MIT License
1.41k stars 188 forks source link

Error reading (native code sln) binlog generated via MSBuild VS2022Preview v143 toolset c++17 #696

Closed andymacx closed 11 months ago

andymacx commented 11 months ago

Error when reading the file: System.ArgumentException: Invalid DateTimeKind value. Parameter name: kind at System.DateTime..ctor(Int64 ticks, DateTimeKind kind) at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsReader.ReadDateTime() in C:\projects\msbuildstructuredlog\src\StructuredLogger\BinaryLogger\BuildEventArgsReader.cs:line 1257 at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsReader.ReadBuildEventArgsFields(Boolean readImportance) in C:\projects\msbuildstructuredlog\src\StructuredLogger\BinaryLogger\BuildEventArgsReader.cs:line 913 at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsReader.ReadProjectStartedEventArgs() in C:\projects\msbuildstructuredlog\src\StructuredLogger\BinaryLogger\BuildEventArgsReader.cs:line 444 at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsReader.Read() in C:\projects\msbuildstructuredlog\src\StructuredLogger\BinaryLogger\BuildEventArgsReader.cs:line 122 at Microsoft.Build.Logging.StructuredLogger.BinLogReader.Replay(Stream stream, Progress progress) in C:\projects\msbuildstructuredlog\src\StructuredLogger\BinaryLogger\BinLogReader.cs:line 98

KirillOsenkov commented 11 months ago

Could you please email me the broken binlog to kirillo microsoft dot com?

andymacx commented 11 months ago

Somewhat interestingly, this only happens with Debug binlog, Release binlog doesn't show this behaviour.

KirillOsenkov commented 11 months ago

Thanks, I got your mail, will look tomorrow.

Meanwhile, try seeing if it still reproduces with the latest version of the viewer 2.1.844), it was published earlier today.

andymacx commented 11 months ago

Appears to be working now! Thanks.

On Wed, 26 Jul 2023 at 07:47, Kirill Osenkov @.***> wrote:

Thanks, I got your mail, will look tomorrow.

Meanwhile, try seeing if it still reproduces with the latest version of the viewer 2.1.844), it was published earlier today.

— Reply to this email directly, view it on GitHub https://github.com/KirillOsenkov/MSBuildStructuredLog/issues/696#issuecomment-1651078281, or unsubscribe https://github.com/notifications/unsubscribe-auth/BBQHCHOUUB32MHPKRGEZPG3XSC4RTANCNFSM6AAAAAA2XKU2VI . You are receiving this because you authored the thread.Message ID: @.***>

--

This email and any attachments is intended solely for the individual or entity to which it is addressed unless explicitly noted otherwise. This email may contain proprietary, confidential and or legally privileged information. If you have received this email in error, you may not use, copy or disclose the information in this message or any attachment in any form whatsoever. Please notify the sender and then immediately and permanently delete it. The contents of this email may not represent the opinion of Trayport Limited, its subsidiaries or affiliates and it does not constitute a contract or guarantee. Trayport Limited accepts no liability for any use of or reliance on this email by anyone. Emails are not secure and may contain viruses, experience delay or interception and their integrity cannot be guaranteed. Recipients should implement appropriate methods to check for viruses before opening any attachment. Trayport Limited does not accept liability for any damage caused by a software virus. If data privacy or a binding message is desired, recipients should use alternate methods of communication.   

 

Trayport Limited, a company registered in England and Wales.

Company Registration Number: 02769279.

Registered office: 7th Floor, 9 Appold Street, London, EC2A 2AP

VAT number: GB274666173.

 

Any trading activity is conducted with the specific trading venue. Trayport is a software provider of trading solutions and is not a trading venue. Trayport does not arrange investments or provide investment advice.  

 Please click here https://www.trayport.com/legal-notice/ for Trayport's full company information.