Skip to content

C#: REWRITE_DOTNET_RPC_SERVER env-var fallback + symlink-safe project references#8017

Merged
macsux merged 1 commit into
mainfrom
macsux/taipei
Jun 16, 2026
Merged

C#: REWRITE_DOTNET_RPC_SERVER env-var fallback + symlink-safe project references#8017
macsux merged 1 commit into
mainfrom
macsux/taipei

Conversation

@macsux

@macsux macsux commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Adds a REWRITE_DOTNET_RPC_SERVER (and REWRITE_DOTNET_RPC_SERVER_VERSION) environment-variable fallback to CSharpRewriteRpc.Builder so the C# RPC server—a source .csproj or a pre-built dll/exe—can be selected without code changes, supporting local cross-repo development and parallel git worktrees. Adds a Directory.Build.targets that lexically normalizes ProjectReference paths to remove .. segments, preventing NuGet restore races when the SDK tree is consumed through a symlink. Removes the now-unneeded ProduceReferenceAssembly override from OpenRewrite.csproj, and adds EnvServerEntryIntegTest which exercises the env-var fallback when REWRITE_DOTNET_RPC_SERVER is set.

… entry

Allow selecting the C# RPC server (a source .csproj or pre-built dll/exe) and
its version via REWRITE_DOTNET_RPC_SERVER / REWRITE_DOTNET_RPC_SERVER_VERSION
environment variables, so local cross-repo development and parallel git
worktrees can override the embedded tool without code changes. Add a
Directory.Build.targets that normalizes ProjectReference paths to avoid
symlinked-checkout restore races, and an integration test exercising the
env-var fallback.
@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite Jun 16, 2026
@macsux macsux merged commit 9d6c78c into main Jun 16, 2026
1 check passed
@macsux macsux deleted the macsux/taipei branch June 16, 2026 01:00
@github-project-automation github-project-automation Bot moved this from In Progress to Done in OpenRewrite Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant