-
Notifications
You must be signed in to change notification settings - Fork 378
[main] Bump a few package versions #14003
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
Conversation
src/SignCheck/Microsoft.SignCheck/Verification/NupkgVerifier.cs
Outdated
Show resolved
Hide resolved
dd15f74 to
94a8da7
Compare
|
/ping reviewers or anyone in @dotnet/arcade-contrib 😁 |
src/Common/Microsoft.Arcade.Test.Common/Microsoft.Arcade.Test.Common.csproj
Outdated
Show resolved
Hide resolved
- small part of dotnet/dnceng#345 - bring System.Security.Cryptography.Xml v6.0.1 in directly - see dotnet/dnceng#1207 - useful wherever Microsoft.Build.Tasks.Core is used - react to deprecation of `ContentItemCollection.FindItemGroups(...)`
- should not be needed w/ Directory.Packages.props change
c5d2060 to
74febe6
Compare
|
/backport to release/8.0 |
|
Started backporting to release/8.0: https://github.com/dotnet/arcade/actions/runs/6750841711 |
|
@dougbu backporting to release/8.0 failed, the patch most likely resulted in conflicts: $ git am --3way --ignore-whitespace --keep-non-patch changes.patch
Applying: [main] Bump a few package versions - small part of https://github.com/dotnet/dnceng/issues/345 - bring System.Security.Cryptography.Xml v6.0.1 in directly - see https://github.com/dotnet/dnceng/issues/1207 - useful wherever Microsoft.Build.Tasks.Core is used - react to deprecation of `ContentItemCollection.FindItemGroups(...)`
Using index info to reconstruct a base tree...
A Directory.Packages.props
M eng/Versions.props
M src/Common/Microsoft.Arcade.Test.Common/Microsoft.Arcade.Test.Common.csproj
M src/Microsoft.DotNet.ApiCompat/src/Microsoft.DotNet.ApiCompat.csproj
M src/Microsoft.DotNet.ApiCompat/tests/Microsoft.DotNet.ApiCompat.Tests.csproj
M src/Microsoft.DotNet.Arcade.Sdk.Tests/Microsoft.DotNet.Arcade.Sdk.Tests.csproj
M src/Microsoft.DotNet.Arcade.Sdk/Microsoft.DotNet.Arcade.Sdk.csproj
M src/Microsoft.DotNet.ArcadeLogging/Microsoft.DotNet.ArcadeLogging.csproj
M src/Microsoft.DotNet.Build.Tasks.Feed/Microsoft.DotNet.Build.Tasks.Feed.csproj
M src/Microsoft.DotNet.Build.Tasks.Installers/Microsoft.DotNet.Build.Tasks.Installers.csproj
M src/Microsoft.DotNet.Build.Tasks.Packaging/src/Microsoft.DotNet.Build.Tasks.Packaging.csproj
M src/Microsoft.DotNet.Build.Tasks.TargetFramework/src/Microsoft.DotNet.Build.Tasks.TargetFramework.csproj
M src/Microsoft.DotNet.Build.Tasks.VisualStudio.Tests/Microsoft.DotNet.Build.Tasks.VisualStudio.Tests.csproj
M src/Microsoft.DotNet.Build.Tasks.VisualStudio/Microsoft.DotNet.Build.Tasks.VisualStudio.csproj
M src/Microsoft.DotNet.GenAPI/Microsoft.DotNet.GenAPI.csproj
M src/Microsoft.DotNet.GenFacades/Microsoft.DotNet.GenFacades.csproj
M src/Microsoft.DotNet.NuGetRepack/tasks/Microsoft.DotNet.NuGetRepack.Tasks.csproj
M src/Microsoft.DotNet.NuGetRepack/tests/Microsoft.DotNet.NuGetRepack.Tests.csproj
M src/Microsoft.DotNet.PackageTesting/Microsoft.DotNet.PackageTesting.csproj
M src/Microsoft.DotNet.SourceBuild/tasks/Microsoft.DotNet.SourceBuild.Tasks.csproj
M src/Microsoft.DotNet.SourceBuild/tests/Microsoft.DotNet.SourceBuild.Tasks.Tests.csproj
M src/Microsoft.DotNet.VersionTools/tasks/Microsoft.DotNet.VersionTools.Tasks.csproj
Falling back to patching base and 3-way merge...
Auto-merging src/Microsoft.DotNet.VersionTools/tasks/Microsoft.DotNet.VersionTools.Tasks.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.VersionTools/tasks/Microsoft.DotNet.VersionTools.Tasks.csproj
Auto-merging src/Microsoft.DotNet.SourceBuild/tests/Microsoft.DotNet.SourceBuild.Tasks.Tests.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.SourceBuild/tests/Microsoft.DotNet.SourceBuild.Tasks.Tests.csproj
Auto-merging src/Microsoft.DotNet.SourceBuild/tasks/Microsoft.DotNet.SourceBuild.Tasks.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.SourceBuild/tasks/Microsoft.DotNet.SourceBuild.Tasks.csproj
Auto-merging src/Microsoft.DotNet.PackageTesting/Microsoft.DotNet.PackageTesting.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.PackageTesting/Microsoft.DotNet.PackageTesting.csproj
Auto-merging src/Microsoft.DotNet.NuGetRepack/tests/Microsoft.DotNet.NuGetRepack.Tests.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.NuGetRepack/tests/Microsoft.DotNet.NuGetRepack.Tests.csproj
Auto-merging src/Microsoft.DotNet.NuGetRepack/tasks/Microsoft.DotNet.NuGetRepack.Tasks.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.NuGetRepack/tasks/Microsoft.DotNet.NuGetRepack.Tasks.csproj
Auto-merging src/Microsoft.DotNet.GenFacades/Microsoft.DotNet.GenFacades.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.GenFacades/Microsoft.DotNet.GenFacades.csproj
Auto-merging src/Microsoft.DotNet.GenAPI/Microsoft.DotNet.GenAPI.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.GenAPI/Microsoft.DotNet.GenAPI.csproj
Auto-merging src/Microsoft.DotNet.Build.Tasks.VisualStudio/Microsoft.DotNet.Build.Tasks.VisualStudio.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.Build.Tasks.VisualStudio/Microsoft.DotNet.Build.Tasks.VisualStudio.csproj
Auto-merging src/Microsoft.DotNet.Build.Tasks.VisualStudio.Tests/Microsoft.DotNet.Build.Tasks.VisualStudio.Tests.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.Build.Tasks.VisualStudio.Tests/Microsoft.DotNet.Build.Tasks.VisualStudio.Tests.csproj
Auto-merging src/Microsoft.DotNet.Build.Tasks.TargetFramework/src/Microsoft.DotNet.Build.Tasks.TargetFramework.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.Build.Tasks.TargetFramework/src/Microsoft.DotNet.Build.Tasks.TargetFramework.csproj
Auto-merging src/Microsoft.DotNet.Build.Tasks.Packaging/src/Microsoft.DotNet.Build.Tasks.Packaging.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.Build.Tasks.Packaging/src/Microsoft.DotNet.Build.Tasks.Packaging.csproj
Auto-merging src/Microsoft.DotNet.Build.Tasks.Installers/Microsoft.DotNet.Build.Tasks.Installers.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.Build.Tasks.Installers/Microsoft.DotNet.Build.Tasks.Installers.csproj
Auto-merging src/Microsoft.DotNet.Build.Tasks.Feed/Microsoft.DotNet.Build.Tasks.Feed.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.Build.Tasks.Feed/Microsoft.DotNet.Build.Tasks.Feed.csproj
Auto-merging src/Microsoft.DotNet.ArcadeLogging/Microsoft.DotNet.ArcadeLogging.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.ArcadeLogging/Microsoft.DotNet.ArcadeLogging.csproj
Auto-merging src/Microsoft.DotNet.Arcade.Sdk/Microsoft.DotNet.Arcade.Sdk.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.Arcade.Sdk/Microsoft.DotNet.Arcade.Sdk.csproj
Auto-merging src/Microsoft.DotNet.Arcade.Sdk.Tests/Microsoft.DotNet.Arcade.Sdk.Tests.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.Arcade.Sdk.Tests/Microsoft.DotNet.Arcade.Sdk.Tests.csproj
Auto-merging src/Microsoft.DotNet.ApiCompat/tests/Microsoft.DotNet.ApiCompat.Tests.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.ApiCompat/tests/Microsoft.DotNet.ApiCompat.Tests.csproj
Auto-merging src/Microsoft.DotNet.ApiCompat/src/Microsoft.DotNet.ApiCompat.csproj
CONFLICT (content): Merge conflict in src/Microsoft.DotNet.ApiCompat/src/Microsoft.DotNet.ApiCompat.csproj
Auto-merging src/Common/Microsoft.Arcade.Test.Common/Microsoft.Arcade.Test.Common.csproj
CONFLICT (content): Merge conflict in src/Common/Microsoft.Arcade.Test.Common/Microsoft.Arcade.Test.Common.csproj
Auto-merging eng/Versions.props
CONFLICT (content): Merge conflict in eng/Versions.props
CONFLICT (modify/delete): Directory.Packages.props deleted in HEAD and modified in [main] Bump a few package versions - small part of https://github.com/dotnet/dnceng/issues/345 - bring System.Security.Cryptography.Xml v6.0.1 in directly - see https://github.com/dotnet/dnceng/issues/1207 - useful wherever Microsoft.Build.Tasks.Core is used - react to deprecation of `ContentItemCollection.FindItemGroups(...)`. Version [main] Bump a few package versions - small part of https://github.com/dotnet/dnceng/issues/345 - bring System.Security.Cryptography.Xml v6.0.1 in directly - see https://github.com/dotnet/dnceng/issues/1207 - useful wherever Microsoft.Build.Tasks.Core is used - react to deprecation of `ContentItemCollection.FindItemGroups(...)` of Directory.Packages.props left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 [main] Bump a few package versions - small part of https://github.com/dotnet/dnceng/issues/345 - bring System.Security.Cryptography.Xml v6.0.1 in directly - see https://github.com/dotnet/dnceng/issues/1207 - useful wherever Microsoft.Build.Tasks.Core is used - react to deprecation of `ContentItemCollection.FindItemGroups(...)`
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Error: The process '/usr/bin/git' failed with exit code 128Please backport manually! |
|
@dougbu an error occurred while backporting to release/8.0, please check the run log for details! Error: git am failed, most likely due to a merge conflict. |
- small part of dotnet/dnceng#345 - was dotnet#14003 in main - bring System.Security.Cryptography.Xml v6.0.1 in directly - see dotnet/dnceng#1207 - useful wherever Microsoft.Build.Tasks.Core is used - add minimal Directory.Packages.props that only handles ^^ for now - choice to use Central Package Management all up in this branch deferred - react to deprecation of `ContentItemCollection.FindItemGroups(...)`
- small part of dotnet/dnceng#345 - was dotnet#14003 in main but could use Directory.Packages.props there - bring System.Security.Cryptography.Xml v6.0.1 in directly - see dotnet/dnceng#1207 - useful wherever Microsoft.Build.Tasks.Core is used - react to deprecation of `ContentItemCollection.FindItemGroups(...)`
| <PackageVersion Include="System.Composition" Version="$(SystemCompositionVersion)" /> | ||
| <PackageVersion Include="System.IO.Packaging" Version="$(SystemIOPackagingVersion)" /> | ||
| <PackageVersion Include="System.Reflection.Metadata" Version="$(SystemReflectionMetadataVersion)" /> | ||
| <PackageVersion Include="System.Security.Cryptography.Xml" Version="$(SystemSecurityCryptographyXmlVersion)" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please explain why this transitive dependency bump was necessary? What was the old version that came in and in which projects?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v6.0.0 came in before, in every project that referenced Microsoft.Build.Tasks.Core
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see. Did you consider updating the Microsoft.Build packages instead? I see that newer packages don't bring the non-desirable one in anymore. VS 17.8 (which is required by main anyway) already supports .NET 8.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I considered it but hit some source-build issues. we could change the approach but I'm not sure that's worthwhile
- small part of dotnet/dnceng#345 - was #14003 in main but could use Directory.Packages.props there - bring System.Security.Cryptography.Xml v6.0.1 in directly - see dotnet/dnceng#1207 - useful wherever Microsoft.Build.Tasks.Core is used - react to deprecation of `ContentItemCollection.FindItemGroups(...)`
ContentItemCollection.FindItemGroups(...)To double check: