Skip to content
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

Add System.Diagnostics.DiagnosticSource.9.0.0 to SourceBuild #11169

Open
JanProvaznik opened this issue Dec 18, 2024 · 6 comments
Open

Add System.Diagnostics.DiagnosticSource.9.0.0 to SourceBuild #11169

JanProvaznik opened this issue Dec 18, 2024 · 6 comments
Labels
Area: Telemetry Candidates benefiting from telemtry data

Comments

@JanProvaznik
Copy link
Member

We need to add System.Diagnostics.DiagnosticSource.9.0.0 to SourceBuild to enable instrumenting using System.Diagnostics.Activities which can then be used in our VS telemetry or via a hook.

Without the package the MSBuild Telemetry WIP does not compile in CI: https://github.com/dotnet/msbuild/pull/11084/checks?check_run_id=34598645353

@JanProvaznik JanProvaznik added the Area: Telemetry Candidates benefiting from telemtry data label Dec 18, 2024
@baronfel
Copy link
Member

Why is this needed? I thought System.Diagnostics and related types were part of the Base Class Library?

@JanProvaznik
Copy link
Member Author

JanProvaznik commented Dec 19, 2024

It's in .NET 9, but not in .NET Standard 2.0 . We target standard in Microsoft.Build.Framework

@JanKrivanek
Copy link
Member

There is a 8.0 already:

https://github.com/dotnet/source-build-reference-packages/tree/main/src/referencePackages/src/system.diagnostics.diagnosticsource

It looks like the 9.0 version will anyway eventually be needed for FSharp

@MichaelSimons - can we request this to be added (System.Diagnostics.DiagnosticSource.9.0.0), or should we just self service produce a PR?

@MichaelSimons
Copy link
Member

Is #11084 going to flow into a 9.0 sdk? If so you can't add the a current assembly for that branch into SBRP. You would simply need to declare a dependency in Version.Details.xml. This will allow source-build to use the live built version of the dependency when building msbuild.

@JanKrivanek
Copy link
Member

So basicaly just?:

darc add-dependency --name System.Diagnostics.DiagnosticsSource -v 9.0.0 -t product -r https://github.com/dotnet/runtime

This is btw. for the VNext (10.0) - no plans to flow to 9.0 sdk

@MichaelSimons
Copy link
Member

This is btw. for the VNext (10.0) - no plans to flow to 9.0 sdk

Then in that case, yes, it should just be added to SBRP. The repo is self-service. The instructions to add new packages is here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Telemetry Candidates benefiting from telemtry data
Projects
None yet
Development

No branches or pull requests

4 participants