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

[main] Update dependencies from dotnet/source-build-externals #47376

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Mar 8, 2025

This pull request updates the following dependencies

From https://github.com/dotnet/source-build-externals

@edvilme
Copy link
Member

edvilme commented Mar 9, 2025

Hello @dotnet/source-build is there a reason source build is failing?

@edvilme edvilme force-pushed the darc-main-bae9d4f9-ae6c-4c66-9e85-c6c7e5472684 branch from e3ec954 to 7d57893 Compare March 9, 2025 20:18
@akoeplinger
Copy link
Member

akoeplinger commented Mar 10, 2025

The reason it's failing is because of this:

[xUnit.net 00:00:01.16]     Microsoft.DotNet.SourceBuild.Tests.SymbolsTests.VerifySdkSymbols [FAIL]
[xUnit.net 00:00:01.17]       Assert.True() Failure
[xUnit.net 00:00:01.17]       Expected: True
[xUnit.net 00:00:01.17]       Actual:   False
[xUnit.net 00:00:01.17]       Stack Trace:
[xUnit.net 00:00:01.17]         /_/test/Microsoft.DotNet.SourceBuild.Tests/SymbolsTests.cs(57,0): at Microsoft.DotNet.SourceBuild.Tests.SymbolsTests.VerifySdkSymbols()
[xUnit.net 00:00:01.17]            at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
[xUnit.net 00:00:01.17]            at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)
[xUnit.net 00:00:01.17]       Output:
[xUnit.net 00:00:01.17]         Executing: tar xzf /__w/1/vmr/artifacts/assets/Release/dotnet-symbols-sdk-10.0.100-preview.3.25160.1-centos.9-x64.tar.gz -C /__w/1/vmr/artifacts/bin/Microsoft.DotNet.SourceBuild.Tests/Release/SymbolsTests/symbols
[xUnit.net 00:00:01.17]         Did not find PDBs for the following SDK files:
[xUnit.net 00:00:01.17]         /__w/1/vmr/artifacts/obj/extracted-dotnet-sdk/sdk/10.0.100-preview.3.25160.1/DotnetTools/dotnet-format/Microsoft.VisualStudio.SolutionPersistence.dll
[xUnit.net 00:00:01.17]         /__w/1/vmr/artifacts/obj/extracted-dotnet-sdk/sdk/10.0.100-preview.3.25160.1/DotnetTools/dotnet-watch/10.0.100-preview.3.25160.1/tools/net10.0/any/Microsoft.VisualStudio.SolutionPersistence.dll
[xUnit.net 00:00:01.17]         /__w/1/vmr/artifacts/obj/extracted-dotnet-sdk/sdk/10.0.100-preview.3.25160.1/Microsoft.VisualStudio.SolutionPersistence.dll

but I'm not sure why, the files I've checked do include the Microsoft.VisualStudio.SolutionPersistence.pdb but for some reason it doesn't end up in dotnet-symbols-sdk .tar.gz

@akoeplinger
Copy link
Member

I think what happens is that it's still using the old version of the package from the nuget feed, but using the .pdb from the new version built in source-build-externals. I've cherry-picked the version changes into #47377, let's see if that fixes it.

@akoeplinger akoeplinger requested review from a team as code owners March 10, 2025 12:29
@ViktorHofer
Copy link
Member

@dotnet/source-build please review the Alex's latest commit which adds a new RepositoryReference from msbuild -> SBE.

@akoeplinger
Copy link
Member

akoeplinger commented Mar 10, 2025

hm it didn't help as the test still fails, we probably still need to go with the combined #47377

actually, I think cherry-picking the version bump of the package itself is necessary too, let me try that

@akoeplinger
Copy link
Member

Ok it appears that worked :)

@akoeplinger
Copy link
Member

msbuild started consuming a package from source-build-externals with dotnet/msbuild@adb4394 so adding the RepositoryReference seems correct to me but @dotnet/source-build please confirm.

@edvilme
Copy link
Member

edvilme commented Mar 10, 2025

Hello @akoeplinger @dotnet/source-build sdk-source-build keeps on failing and lists MSBuild as the reason.
Would merging #47377 first help with this?

cc @dotnet/kitten

@akoeplinger
Copy link
Member

Yes I think this will need #47377 which needs dotnet/msbuild#11561 which needs dotnet/source-build-externals#458 😆

@edvilme
Copy link
Member

edvilme commented Mar 10, 2025

Yes I think this will need #47377 which needs dotnet/msbuild#11561 which needs dotnet/source-build-externals#458 😆

Great thanks, will keep an aye :)

dotnet-maestro bot and others added 4 commits March 11, 2025 09:51
…nals build 20250306.1

Microsoft.SourceBuild.Intermediate.source-build-externals
 From Version 10.0.615301 -> To Version 10.0.615601
Co-authored-by: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com>
(cherry picked from commit 5a93642)
…nals build 20250310.1

Microsoft.SourceBuild.Intermediate.source-build-externals
 From Version 10.0.615301 -> To Version 10.0.616001
@edvilme edvilme force-pushed the darc-main-bae9d4f9-ae6c-4c66-9e85-c6c7e5472684 branch from e892f40 to 84f9ee8 Compare March 11, 2025 16:52
@ViktorHofer
Copy link
Member

Still blocked on #47377. Paaaaatieeeence ;)

@edvilme
Copy link
Member

edvilme commented Mar 11, 2025

Still blocked on #47377. Paaaaatieeeence ;)

Ohh sorry, updated the wrong PR hehe

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants