-
Notifications
You must be signed in to change notification settings - Fork 255
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
VS 2017: PackageType=DotnetCliTool fails to install #4190
Comments
If this is helpful, here is the code that throws the exception: https://github.com/NuGet/NuGet.Client/blob/4cccb13833ad29d6a0bcff055460d964f1b49cfe/src/NuGet.Core/NuGet.PackageManagement/InstallationCompatibility.cs#L140-L157 |
@natemcmaster - We punted the work to support tools installs via the UI with the idea that tools will be installed by default with new projects ... and that people can edit the csproj if an addition is needed. |
Makes sense. Would still love to see this make it in to RTM. But if this gets bumped, can we at least change the error? It currently says that the project doesn't support CLI tools, which may mislead some to believe .NET Core CLI tools don't work anymore. |
+1 on a better error message when doing this from the Manage NuGet packages UI Rob suggested it might be possible to get this working from the Install-Package power shell command for RC3 / RTM. @rrelyea ? |
|
This issue still exists with Dot Net Standard 2.0. Is there any update on this issue? I want to be able to use EF Core in a CL targeting standard 2.0 and this stops this from being possible |
As a workaround, you can edit the .csproj file and add the tool manually, e.g.: <ItemGroup>
<DotNetCliToolReference Include="Microsoft.EntityFrameworkCore.Tools.Dotnet" Version="2.0.0-preview2-final" />
</ItemGroup> |
@geirsagberg, |
Same error than @st2forget . I tried to add the package manually in the csproj but the error is still the same. |
It works now. I did nothing, just wait ... |
Still not working for me, same error as @st2forget |
You still install them in the project (into the same CSPROJ file), but as tools references, not package references. They are represented as different XML elements inside the CSPROJ. |
Any news on this? Are we doomed to forever manually edit the .csproj file? |
I think with the upcoming Global Tools feature the priority of this might be a bit lower. I think @KathleenDollard and @richlander can share more thoughts on this. |
Tools aren't really packages, they have a different purpose, so We're working on a better tooling story overall, and EF is definitely in that mix. More to come soon. |
@KathleenDollard are you saying that there is no solution for the current release of the product? |
Still having this issue with the latest SDK. Error when adding the tools via NUGET as well. Right now we have to add that line manually to do migrations etc...
How are people using EF in CORE without this working? |
Because we are doing it manually as well. It's annoying, but it shouldn't block you. |
Here are the docs on how to install the EF Core tools. (manually) |
As specified by @geirsagberg adding the following dependancy directly to the .csproj file worked for me.
|
I'm experiencing the same issue. Someone please fix it. |
We're not planning on adding CLI support for dotnetclitoolreferences anymore. |
@rrelyea What does that mean? Are you going to stop using DotNetCliToolReferences altogether? Or that they won't work when installed via the CLI. Cause this bug was occurring when installed via VS. |
it works for me |
Little help for those trying Core 3 Preview https://wildermuth.com/2019/08/19/My-First-Look-at-ASP-NET-Core-3-0 |
Details
VS 2017: 15.0.26014
NuGet GUI
Worked in NuGet: 3.5.0.1996 with VS 2015 u3
Repro
Fails with error:
More details
The package's NuSpec:
Output window:
The project:
cc @joelverhagen
The text was updated successfully, but these errors were encountered: