Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Update SDK to preview 3 release#100945

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
ViktorHofer merged 5 commits intomainfromMichalStrehovsky-patch-1
Apr 15, 2024
Merged

Conversation

@MichalStrehovsky
Copy link
Member

No description provided.

@ghostghost added the needs-area-labelAn area label is needed to ensure this gets routed to the appropriate area owners labelApr 12, 2024
@MichalStrehovskyMichalStrehovsky added area-Infrastructure and removed needs-area-labelAn area label is needed to ensure this gets routed to the appropriate area owners labelsApr 12, 2024
Comment on lines +14 to +15
<PackageReferenceInclude="Microsoft.AspNetCore.Components.WebAssembly"Version="9.0.0-preview.3.24172.13" />
<PackageReferenceInclude="Microsoft.AspNetCore.Components.WebAssembly.DevServer"Version="9.0.0-preview.3.24172.13"PrivateAssets="all" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Do we have an issue that tracks removing these hardcodes?

Copy link
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Do we have an issue that tracks removing these hardcodes?

Might want to ask that@radekdoulik@maraf@pavelsavara who were involved in the PR that introduced it; looks like this has been causing extra work when bumping the SDK since 8.0 RC1.

@MichalStrehovskyMichalStrehovsky added the NO-MERGEThe PR is not ready for merge yet (see discussion for detailed reasons) labelApr 15, 2024
@MichalStrehovsky
Copy link
MemberAuthor

Looks like the new run is hitting a hard crash in the runtime on osx-x64. I doubt it was caused by the latest commit, so this is one of those fun non-deterministic crashes in MSBuild:

2024-04-15T09:34:59.7728930Z     /Users/runner/work/1/s/dotnet.sh: line 27: 91742 Segmentation fault: 11  ${dotnetPath} "$@"2024-04-15T09:34:59.8872770Z /Users/runner/work/1/s/src/tests/JIT/HardwareIntrinsics/Arm/Directory.Build.targets(15,5): error MSB3073: The command ""/Users/runner/work/1/s/dotnet.sh" build /Users/runner/work/1/s//src/tests/Common/GenerateHWIntrinsicTests/GenerateHWIntrinsicTests_Arm.csproj -c Release --no-restore /p:IntermediateOutputPath=/Users/runner/work/1/s/artifacts/tests/coreclr/obj/osx.x64.Release/Managed/JIT/HardwareIntrinsics/Arm/Sve/Sve_r//GenerateHWIntrinsicTests/ /p:OutputPath=/Users/runner/work/1/s/artifacts/tests/coreclr/osx.x64.Release/JIT/HardwareIntrinsics/Arm/Sve/Sve_r//GenerateHWIntrinsicTests/" exited with code 139. [/Users/runner/work/1/s/src/tests/JIT/HardwareIntrinsics/Arm/Sve/Sve_r.csproj] [/Users/runner/work/1/s/src/tests/build.proj]

I guess this is now NO-MERGE/blocked. Filed#101049.

@akoeplinger
Copy link
Member

akoeplinger commentedApr 15, 2024
edited
Loading

Doesn't seem related given we found more hits in older releases in#101049 so I triggered a rerun.

am11 reacted with thumbs up emoji

@am11
Copy link
Member

All green after the rerun.:shipit:

@ViktorHoferViktorHofer merged commitdd7c82e intomainApr 15, 2024
@ViktorHoferViktorHofer deleted the MichalStrehovsky-patch-1 branchApril 15, 2024 16:08
matouskozak pushed a commit to matouskozak/runtime that referenced this pull requestApr 30, 2024
* Update global.json* Update blazor.csproj* Delete useless `RuntimeIdentifiers`* Fix up illinker tests* Fix two more preview1 versions---------Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsMay 16, 2024
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@akoeplingerakoeplingerakoeplinger approved these changes

@ViktorHoferViktorHoferViktorHofer approved these changes

@lewinglewingAwaiting requested review from lewinglewing is a code owner

@pavelsavarapavelsavaraAwaiting requested review from pavelsavarapavelsavara is a code owner

@marek-safarmarek-safarAwaiting requested review from marek-safarmarek-safar is a code owner

@marafmarafAwaiting requested review from marafmaraf is a code owner

@ilonatommyilonatommyAwaiting requested review from ilonatommyilonatommy is a code owner

Labels

area-InfrastructureNO-MERGEThe PR is not ready for merge yet (see discussion for detailed reasons)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@MichalStrehovsky@akoeplinger@am11@ViktorHofer

[8]ページ先頭

©2009-2025 Movatter.jp