-
Notifications
You must be signed in to change notification settings - Fork 10k
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
Update .NET SDK to 10.0.100-alpha.1.24553.6 #58776
Conversation
Update .NET SDK to version 10.0.100-alpha.1.24553.6. --- updated-dependencies: - dependency-name: Microsoft.NET.Sdk dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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.
undo submodules?
Sorry, this is my git client, I usually catch it in time. Will fix. |
b5d1656
to
f30edff
Compare
This is the result of https://github.com/dotnet/runtime/pull/109467/files However XUnit should handle that now, so I will "try" (not even sure that's possible) to update the version we are using. |
The same template tests are failing on both windows and linux @javiercn anybody we can ask for help? |
@@ -29,7 +29,7 @@ public void Encrypt_Decrypt_RoundTrips() | |||
byte[] decipheredtext = encryptor.Decrypt(new ArraySegment<byte>(ciphertext), aad); | |||
|
|||
// Assert | |||
Assert.Equal(plaintext, decipheredtext); | |||
Assert.Equal(plaintext.AsSpan(), decipheredtext.AsSpan()); |
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.
@sebastienros - why is this change necessary now? Both expected and actual are byte[]
. Why do they need to be cast to span?
7a8ae19
to
0f9d339
Compare
@dotnet/source-build - the pre-built leg is failing:
Any guidance on how to address this? |
I'm pretty sure this requires that we have a .NET 9.0 targeting pack defined in SBRP. But we're going to want to wait until after this 10.0 SDK gets updated to reference the 9.0 GA version. |
This can be marked as an allowed prebuilt IMO for the time being until we get the 9.0 GA targeting packs in SBRP. |
Looks like the project template test failures are caused by static web asset failures. I was able to repro this locally by running the project template tests:
cc @javiercn |
@MackinnonBuck thanks. I'll take a look at it tomorrow first thing in the morning. |
I'm getting the following error. Any idea how to get past it?
|
@MackinnonBuck can you get me the binlog? |
@@ -122,11 +122,13 @@ | |||
<GenerateStaticWebAssetEndpointsPropsFile | |||
StaticWebAssets="@(V4Assets)" | |||
StaticWebAssetEndpoints="@(V4AssetEndpoints)" | |||
PackagePathPrefix="staticwebassets/V4" |
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.
Do we know what changed to break this? Is this a sign of some underlying bug?
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 don't know yet, but I plan to look at it on Monday, at a quick glance, the endpoints in the package are incorrect too (we will have to service that).
It doesn't currently fail I think because we have a process to "upgrade" older packages and define endpoints and the endpoints end up being generated as part of the user build.
Not sure what the difference is between the build in the repo and the end user build that causes it to fail here. In the end, it's a bug in the package.
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.
Filed an issue to track
Superseded by #58862. |
I think we should restore this branch and get it in a mergeable state, otherwise we'll need to apply the same changes (and there are lots of them) in the new PR. We can keep the other PR and it will hopefully just work when this one is in. @eerhardt @MackinnonBuck |
Agreed. I suggest closing the new PR and re-open this PR. |
We can't reopen the original one because the branch got deleted |
The branch was recreated (or force pushed) I believe by Eric. We can see it: https://github.com/dotnet/aspnetcore/branches/all?query=update-dotnet-sdk-10.0.100-alpha.1.24553.6&lastTab=overview We may just need a new PR |
Updates the .NET SDK to version
10.0.100-alpha.1.24553.6
, which also updates the .NET runtime from version10.0.0-alpha.1.24507.22
to version10.0.0-alpha.1.24552.2
.This pull request was auto-generated by GitHub Actions.