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

Block package reference to tool package type when there is no property to indicate the project is a temporary project to restore the package #6198

Closed
wli3 opened this issue Nov 20, 2017 · 3 comments
Assignees
Labels
Area:RestoreTool V1/V2 tool restore Type:DCR Design Change Request

Comments

@wli3
Copy link

wli3 commented Nov 20, 2017

Tools CLI ask

tool package can reference normal package

tool package can reference tool package

normal package cannot reference tool package

rid split requires

 mytool
 (
identity package, tool package) -> win-x64.mytool (rid specific package, a tool package)
                                         
                                         ->linux-x64.mytool (rid specific package, a tool package) 
                                   
                                          -> Microsoft.NETCore.Platform (a normal package to provide rid relation)
@wli3 wli3 mentioned this issue Nov 20, 2017
4 tasks
@PatoBeltran PatoBeltran added the Type:DCR Design Change Request label Nov 21, 2017
@nkolev92 nkolev92 added the Area:RestoreTool V1/V2 tool restore label Dec 4, 2017
@nkolev92 nkolev92 added this to the 4.6 - Dec26-Jan13 milestone Dec 4, 2017
@nkolev92 nkolev92 self-assigned this Dec 4, 2017
@nkolev92
Copy link
Member

nkolev92 commented Jan 5, 2018

Due to the way we do compat checking and performance considerations, we will be doing the follwoing:

ToolPackage cannot be found in the normal project graph (only in the special project graph).
Top level dependencies of DotnetToolReference projects HAVE to be of DotnetTool type!

So in theory a package can reference a tool project, but it will never be restoreable.
In addition, people would have to do crazy workarounds to build a package like this.

@bricelam
Copy link

bricelam commented Jan 5, 2018

ToolPackage cannot be found in the normal project graph

I'm arguing semantics here, but shouldn't this be: Only Dependency packages are allowed

@nkolev92
Copy link
Member

Closed in NuGet/NuGet.Client#1900

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area:RestoreTool V1/V2 tool restore Type:DCR Design Change Request
Projects
None yet
Development

No branches or pull requests

4 participants