-
Notifications
You must be signed in to change notification settings - Fork 252
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
NuGet pack should reject duplicated entries in the same directory #9939
Comments
@ericstj did the investigation and can comment on how the |
Related change is here: dotnet/arcade#5818 Essentially we found that we had created NuPkg files with duplicate files and had to work to undo this. Such packages are essentially broken and can't be reliably used. Tooling that operates at the ZIP level don't expect to see this and ended up failing in our build pipeline. This change is us undoing that mistake. |
This change regressed this in Arcade: dotnet/arcade@50da2e6#r40426132 And it was fixed in the Arcade currently in use in rc1 in windowsdesktop: dotnet/arcade@b1754cc |
I think this actually may be because windowsdesktop was creating its nuget packages using a non-standard process, but @zkat we should still ensure that you can't get into this situation using the standard usage paths. @jaredpar said he had seen similar issues with roslyn, which I believe does use the standard pack processes. |
Closing as dupe, and assigning that one to myself instead. |
Please read the following information before posting the issue.
Details about Problem
NuGet product used (NuGet.exe | VS UI | Package Manager Console | dotnet.exe): Any nuget path that can pack.
NuGet version (x.x.x.xxx): Any recent nuget version.
Detailed repro steps so we can see the same problem
It is possible to create a package that contains multiple files with the same name in the same directory. This cannot be unpacked on any operating system in a predictable manner, and a push to nuget.org fails.
The text was updated successfully, but these errors were encountered: