Skip to content

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jul 30, 2025

Backport of #118120 to release/9.0-staging

/cc @steveisok @lateralusX

Customer Impact

  • Customer reported
  • Found internally

On the MacOS 26, we found the debugger startup handshake using named sempahores are now broken when the debugger and the debuggee are signed with different team id's. This casues the entire VS Code C# debugging experience to be non-functional.

Interestingly, the MacOS release notes only show this being depreicated on iOS. Perhaps this will end up being true on future betas, but I don't think we should hold out any hope.

The fix in this PR uses named pipes first in the handshake and will fall back to the previous implementation if necessary. Mostly on backwards compatibility grounds.

Regression

  • Yes
  • No

[If yes, specify when the regression was introduced. Provide the PR or commit if known.]

Testing

Manual before broken / after fixed testing on VSCode.

Risk

Low

IMPORTANT: If this backport is for a servicing release, please verify that:

  • The PR target branch is release/X.0-staging, not release/X.0.

Package authoring no longer needed in .NET 9

IMPORTANT: Starting with .NET 9, you no longer need to edit a NuGet package's csproj to enable building and bump the version.
Keep in mind that we still need package authoring in .NET 8 and older versions.

@steveisok steveisok added the Servicing-consider Issue for next servicing release review label Aug 4, 2025
Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

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

lgtm. please get a code review. we will take for consideration in 9.0.x

@jeffschwMSFT jeffschwMSFT added this to the 9.0.x milestone Aug 4, 2025
@steveisok steveisok requested a review from a team August 4, 2025 21:26
@rbhanda rbhanda modified the milestones: 9.0.x, 9.0.9 Aug 5, 2025
@jeffschwMSFT jeffschwMSFT added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Aug 5, 2025
@steveisok steveisok enabled auto-merge (squash) August 5, 2025 17:56
@steveisok
Copy link
Member

/ba-g space issue unrelated to change

1 similar comment
@steveisok
Copy link
Member

/ba-g space issue unrelated to change

@hoyosjs
Copy link
Member

hoyosjs commented Aug 5, 2025

/ba-g insufficient space error.

@hoyosjs hoyosjs disabled auto-merge August 5, 2025 21:04
@hoyosjs
Copy link
Member

hoyosjs commented Aug 5, 2025

Bypassing - build analysis is stuck.

@hoyosjs hoyosjs merged commit df8f6a5 into release/9.0-staging Aug 5, 2025
92 of 98 checks passed
@hoyosjs hoyosjs deleted the backport/pr-118120-to-release/9.0-staging branch August 5, 2025 21:04
@github-actions github-actions bot locked and limited conversation to collaborators Sep 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants