Skip to content

Conversation

@dotnet-maestro
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

@ghost ghost added Area-Infrastructure untriaged Request triage from a team member labels Mar 8, 2025
@edvilme
Copy link
Contributor

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
Contributor

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
Contributor

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
Contributor

edvilme commented Mar 11, 2025

Still blocked on #47377. Paaaaatieeeence ;)

Ohh sorry, updated the wrong PR hehe

@akoeplinger akoeplinger enabled auto-merge (squash) March 11, 2025 20:49
@akoeplinger akoeplinger merged commit bcd9475 into main Mar 12, 2025
42 of 43 checks passed
@akoeplinger akoeplinger deleted the darc-main-bae9d4f9-ae6c-4c66-9e85-c6c7e5472684 branch March 12, 2025 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-CodeFlow untriaged Request triage from a team member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants