-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
netcoredbg: init at 1.2.0-825 #103940
netcoredbg: init at 1.2.0-825 #103940
Conversation
I'm looking forward to this addition. Great work so far! |
I was having some trouble getting it to build with .NET 5. I'm going to try again soon, maybe later today. |
e8246fc
to
18707ef
Compare
I've updated this to the latest netcoredbg version, .NET 5, and addressed the review. |
You could get the runtime version with something like this, if that's the problem: dotnet --list-runtimes | grep -Po '^Microsoft.NETCore.App \K.*?(?= )' |
Updated to latest netcoredbg and improved packaging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few minor comments, but this mostly looks good :)
@ofborg eval |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few more comments :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some final nits, though functionality wise this looks good to me 👍
Result of nixpkgs-review
run on x86_64-linux
1 package built:
- netcoredbg
The binary also works perfectly as expected :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the work!
Motivation for this change
netcoredbg is an open-source debugger for .NET Core applications. The most popular one, vsdbg, is a binary blob distributed with the Visual Studio Code C# extension. However, vsdbg's license is incredibly restrictive. netcoredbg is a drop-in replacement, and is also compatible with other editors.
Note that the package here is... kind of awful. This is both because of the fact that netcoredbg uses CMake in pretty strange ways, and because packaging .NET Core applications in general is pretty bad at the moment.
Closes #103938
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)