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.

Sign up for GitHub

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

Jump to bottom

Show error when using nuget.exe to pack SDK csproj #7778

Closed
Tracked by #6285
loic-sharma opened this issue Feb 10, 2019 · 5 comments · Fixed by NuGet/NuGet.Client#4795
Closed
Tracked by #6285

Show error when using nuget.exe to pack SDK csproj #7778

loic-sharma opened this issue Feb 10, 2019 · 5 comments · Fixed by NuGet/NuGet.Client#4795
Assignees
@nkolev92
Labels
Category:SeasonOfGiving https://devblogs.microsoft.com/nuget/nuget-season-of-giving/#season-of-giving Functionality:Pack Priority:2 Issues for the current backlog. Product:NuGet.exe NuGet.exe Type:DCR Design Change Request
Milestone
6.5

Comments

@loic-sharma
Copy link
Contributor

loic-sharma commented Feb 10, 2019 •
edited by nkolev92
Loading

According to this and this discussions, NuGet.exe is not supported for packing SDK projects. The current experience is confusing as it seems like NuGet.exe pack "works" but ignores properties set in the project. Instead, NuGet.exe should show a helpful error message that guides users to use dotnet.exe instead.

Design - #12088
@rrelyea rrelyea added this to the Backlog milestone Feb 12, 2019
@IanKemp IanKemp mentioned this issue Sep 2, 2019
Closed
@IanKemp
Copy link

IanKemp commented Sep 2, 2019

This is the only logical solution, since the NuGet team apparently isn't interested in adding critical functionality to their app - merely making it look like it works when it actually doesn't, then ignoring the fact that it doesn't actually work for the next two-and-a-half years.

@0xced 0xced mentioned this issue Sep 20, 2019
Closed
0xced added a commit to 0xced/docs.microsoft.com-nuget that referenced this issue Sep 23, 2019
@0xced
Do not document using nuget pack with a csproj
5f61fff 
It is not supported and doesn't work, see [Show error when using nuget.exe to pack SDK csproj](NuGet/Home#7778)
@0xced 0xced mentioned this issue Sep 23, 2019
Closed
@loic-sharma
Copy link
Contributor Author

loic-sharma commented Sep 24, 2019 •
edited
Loading

@rrelyea @anangaur Thoughts? This is a "breaking" change but would greatly help newcomers. For example: NuGet/docs.microsoft.com-nuget#1700

@aortiz-msft aortiz-msft added the Category:Quality Week Issues that should be considered for quality week label Feb 13, 2020
@stefanluchian
Copy link

The problem with "nuget pack" is known since 2017.
One could do the trick with "msbuild -t:pack".
One could use "dotnet pack".
But the others that rely on "nuget pack" loose many hours till they figure out what is going on.

@microsoft: Will you please do something about it?

@nkolev92 nkolev92 removed this from the Backlog milestone Apr 16, 2020
@nkolev92 nkolev92 added the Priority:2 Issues for the current backlog. label Apr 16, 2020
@aortiz-msft
Copy link
Contributor

@zkat - #4491, which points to this issue, has 47 upvotes. Might want to consider for Sprint 179.

@zkat zkat mentioned this issue Jul 1, 2021
Open
18 tasks
@aortiz-msft aortiz-msft added Type:DCR Design Change Request and removed Category:Quality Week Issues that should be considered for quality week labels Feb 2, 2022
@nkolev92 nkolev92 added Category:SeasonOfGiving https://devblogs.microsoft.com/nuget/nuget-season-of-giving/#season-of-giving and removed Pipeline:Backlog labels Aug 29, 2022
@nkolev92 nkolev92 self-assigned this Sep 6, 2022
@nkolev92 nkolev92 mentioned this issue Sep 9, 2022
Merged
@nkolev92
Copy link
Member

nkolev92 commented Sep 9, 2022

Proposal for this change is out in #12088.

@nkolev92 nkolev92 mentioned this issue Sep 14, 2022
Merged
8 tasks
@martinrrm martinrrm added this to the 6.5 milestone Feb 9, 2023
p0onage added a commit to p0onage/docs.microsoft.com-nuget that referenced this issue Mar 2, 2023
@p0onage
Update cli-ref-pack.md
9b0956b 
Updated documentation to show depreciated pack command on Nuget CLI 

Show error when using nuget.exe to pack SDK csproj #7778
NuGet/Home#7778
@p0onage p0onage mentioned this issue Mar 2, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nkolev92 nkolev92

Labels
Category:SeasonOfGiving https://devblogs.microsoft.com/nuget/nuget-season-of-giving/#season-of-giving Functionality:Pack Priority:2 Issues for the current backlog. Product:NuGet.exe NuGet.exe Type:DCR Design Change Request
Projects
None yet
Milestone
6.5
8 participants
@loic-sharma @IanKemp @nkolev92 @rrelyea @martinrrm @heng-liu @aortiz-msft @stefanluchian