-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
base: main
Are you sure you want to change the base?
[main] Update dependencies from dotnet/source-build-externals #47376
Conversation
Hello @dotnet/source-build is there a reason source build is failing? |
e3ec954
to
7d57893
Compare
The reason it's failing is because of this:
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 |
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. |
@dotnet/source-build please review the Alex's latest commit which adds a new RepositoryReference from msbuild -> SBE. |
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 |
Ok it appears that worked :) |
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. |
Hello @akoeplinger @dotnet/source-build cc @dotnet/kitten |
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 :) |
…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
e892f40
to
84f9ee8
Compare
Still blocked on #47377. Paaaaatieeeence ;) |
Ohh sorry, updated the wrong PR hehe |
This pull request updates the following dependencies
From https://github.com/dotnet/source-build-externals