fix: Remove RuntimeIdentifiers from .csproj to fix NuGet tool installation #89
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
The DotnetToolSettings.xml Version="2" format was preventing installation on .NET SDK 9.0.306 (requires 9.0.400+). This issue was introduced in PR #87 when the publish loop was removed from pack.sh.
Root Cause
When
dotnet packruns without--no-build, it sees<RuntimeIdentifiers>in the .csproj and generates DotnetToolSettings.xml Version="2" format, which requires .NET SDK 9.0.400+.Solution
Remove
<RuntimeIdentifiers>property from all tool projects. The RIDs are only needed for self-contained publishing, where they're specified via-r <RID>flag in publish-self-contained.sh.Changes
<RuntimeIdentifiers>from cycod.csproj, cycodt.csproj, cycodmd.csproj, cycodgr.csprojTesting
? Built and packed locally with version 1.0.0-alpha-20251222.2
? Verified DotnetToolSettings.xml uses Version="1"
? Successfully installed all tools:
dotnet tool install -g <tool> --version 1.0.0-alpha-20251222.2? Verified all tools run with correct version
Fixes
Installation error: