-
Notifications
You must be signed in to change notification settings - Fork 526
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
Improve CLI Tools support on solution level #2497
Comments
Ok I think I have another idea (maybe a bit crazy). Assume that for every cli tool we generate wrapper scripts in .nugetcache/.clitools the only requirement would be to have ".nugetcache/.clitools" in path. Do you think this could work? *note we could have another indirection to call the correct paket verion before calling the tool |
related to #2919 as a note, dotnetclitool may expect some env var to be set (like msbuild exe etc) because are needed to inspect the project file in some tools. so wrapper should also mimic that |
CLI tools are already obsolete. |
Now that we have dotnet cli tool support, we could improve that.
This is one possible root to enable CLI Tools on the solution level, possibly conflicting with #2249
*.deps.json
https://github.com/dotnet/cli/blob/master/Documentation/specs/runtime-configuration-file.mdpaket-files/.bin/compile-fsc
paket-files/.bin/compile-fsc.cmd
PATH="./paket-files/.bin:$PATH"
or
SET PATH=paket-files\.bin;%PATH%
-> People can use dotnet tools directly on solution level for example
compile-fcs --help
Why is this conflicting with #2249? Because we could probably get away with generating
project.assets.json
as well instead of*.deps.json
. But imho we should skipassets.json
if possible. That file is owned by nuget whiledeps.json
is owned by the runtime alone.This would be a first step to simplify
dotnet fable yarn-run start
intofable yarn-run start
.Question is can we improve that even further to make it work on any folder within the solution folder?
Is this a way forward? What do you think?
/cc @enricosada @forki
The text was updated successfully, but these errors were encountered: