PR #640 causes very annoying .NET SDK not found
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 86/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- csharp
- Domain
- build-system, developer-experience
Research direction
Start with global.json and compare its SDK rollForward and paths settings with the workaround described in the issue; review build.ps1 and build.sh to understand the local SDK directories they install. Verify the solution loads in Rider/VS with only SDK 10.0.400 installed, and confirm CI still works with and without a pre-installed matching SDK.
Written by the indexing model from the issue text.
Description
Usage Information
Fallout as developer (not consumer) via Rider
Description
The PR #640 causes a very annoying "bug". It does resist to load the solution if 10.0.302 SDK isn't installed.
Running .\build.ps1 installs it into .fallout/temp/dontet-{win,unix}, but this does not help Rider. Of course you could change in the Rider settings the SDK search path, but you have to do this every SDK update.
Reproduction Steps
- Having .NET SDK 10.0.400 installed and no 10.0.3xx
- Checkout develop
Expected Behavior
Solution loads in Rider/VS
Actual Behavior
Does not load
Regression?
You could argue: yes (because rollForward: latestMinor got this)
Known Workarounds
- Changing rollForward back to "latestMinor"
- Update global.json to this content:
{
"sdk": {
"version": "10.0.302",
"rollForward": "latestPatch",
"paths": [
".fallout/temp/dotnet-win",
".fallout/temp/dotnet-unix",
"$host$"
]
}
}
This tells Rider/VS/IDE to look into the locally install directories first and then what's installed on the host.
Works locally, in CI/CD with pre-installed SDK and in CI/CD with no matching SDK installed (since build.{sh,ps1} installs it and will be picked up)
I propose to add this to the global json, since it doesn't hurt anyone, but restores a fallback path for local dev.
Could you help with a pull-request?
Yes
- Dominant language
- C#
- Stars
- 158
- Forks
- 19
- Avg merge
- 5d 2h
- Merged PRs (30d)
- 10
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Fallout-build/Fallout
-
bug needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Fallout-build/Fallout#670 ·
-
CD enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Fallout-build/Fallout#649 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Fallout-build/Fallout#630 · 1 comment · 1 reaction ·
-
enhancement target/backlog
Difficulty 1/5 Under an hour Newbie friendliness 68/100
Fallout-build/Fallout#609 ·
-
enhancement target/backlog
Difficulty 1/5 Under an hour Newbie friendliness 72/100
Fallout-build/Fallout#608 ·
All issues in Fallout-build/Fallout
Similar issues
-
type/automation type/tech-debt
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
t/bug
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
ci-failure-cause test-failure
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
area:auth FE mvp P3
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
klasolsson81/jobbliggaren#1788 ·