Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version 5-beta crashes with nullref exception #1170

Closed
OsirisTerje opened this issue May 6, 2024 · 5 comments
Closed

Version 5-beta crashes with nullref exception #1170

OsirisTerje opened this issue May 6, 2024 · 5 comments
Assignees
Milestone

Comments

@OsirisTerje
Copy link
Member

OsirisTerje commented May 6, 2024

Trying to run the dll directly:
image

using dotnet run
image

I believe these two have the same root cause, but the message from the latter may be misleading.

@OsirisTerje OsirisTerje changed the title Version 5 crashes with nullref exception Version 5-beta crashes with nullref exception May 6, 2024
@Evangelink
Copy link
Contributor

@OsirisTerje As discussed offline, I'll take this one!

@Evangelink
Copy link
Contributor

Fixed by #1171

@Evangelink
Copy link
Contributor

dotnet run issue is still present but doing dotnet run --project solves the issue. I have reached out to some people to understand why the "default" gesture isn't working.

@Evangelink
Copy link
Contributor

I have talked with the team and checked the docs https://learn.microsoft.com/dotnet/core/tools/dotnet-run and this syntax isn't valid. We should either run dotnet run from a folder, and it would execute only if a single project is found (see https://github.com/dotnet/sdk/blob/171723b0d76e60d147d259140adbfd078aa252a1/src/Cli/dotnet/commands/dotnet-run/RunCommand.cs#L294-L305) or dotnet run --project <path-to-project>.

@OsirisTerje
Copy link
Member Author

OsirisTerje commented May 7, 2024

Ok, it finds both the sln and the csproj. Normally they are never in the same folder, so this makes sense. I tested by removing the sln file, and then it works.
So I'll move the sln to one level up, and use that for both csprojs, for use with VS. Then people can test using simple dotnet run inside each folder

@OsirisTerje OsirisTerje added this to the 5.0 milestone May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants