-
Notifications
You must be signed in to change notification settings - Fork 1.1k
--create-manifest-if-needed for tool manifest behaviour discussion, and the -d option to list location searched doesn't do what it claims #36830
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
Comments
Thank you for the suggestion! This is a great suggestions and we'd love to take that into consideration. However, we are also trying to avoid that there could potentially generating manifest file accidentally on unexpected locations, and you could run unexpected tools, as the side effect of creating a manifest file by default. We did take it into consideration when adding the |
Adding another outcome I encounter, and reinsures me a tiny bit in my reading comprehension
$ dotnet tool install -d fsharp-analyzers
Unrecognized command or argument 'fsharp-analyzers'.
Description:
Install global or local tool. Local tools are added to manifest and restored.
... all the help |
I feel like this should be prioritized, but I dont understand the side effects of creating the tool manifest. |
Seems like a dupe of #15254 to some extent (though this is more about the -d flag), though that issue was written before this flag existed. /baronfel remarked on how to write this there. #15254 (comment) |
Triage: we probably want to change the message to suggest a different way of getting diagnostics data as -d is debug and doesn't list those locations currently. |
Triage:
|
I've not been able to try it with latest rc2, but wondering if the command still fails due to no manifest, is it advertising the option implemented in #31231?
this should be the default (if --create-manifest-if-needed is still explicit opt-in as default, rather than opt out), if it is opt-out it should just go ahead and create the manifest and print the location of it when creating it.
Also, when I run this command in https://github.com/stephan-tolksdorf/fparsec/
I get:
the
-d
flag doesn't make it list anything, only a more detailed stack trace.I tried before and after removing global.json from the fparsec repository.
The text was updated successfully, but these errors were encountered: