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

Variations in NuGet Package ID Case Sensitivity can lead to an Editor Restart Loop on Install #704

Open
ilexp opened this issue Dec 9, 2018 · 0 comments
Labels
Bug It's broken and should be fixed Editor Area: Duality editor or support libraries Nice2Have Beneficial, but only very slightly so
Milestone

Comments

@ilexp
Copy link
Member

ilexp commented Dec 9, 2018

Summary

In some rare cases, NuGet packages can have inconsistent naming, where package ID can differ in case sensitivity when retrieved / viewed on the web, vs. when downloaded and installed. To Duality, the installed package will then be a different one than the online package, causing an editor restart loop after installation.

How to reproduce

@SirePi initially reproduced this with an earlier version of the YAUI plugin by using the $id variable in the .nuspec, switching from CamelCase .Core to (derived from the .csproj) lowercase .core, triggering the issue.

notepad _2018-12-09_21-09-06
explorer_2018-12-09_21-09-44
nugetpackageexplorer_2018-12-09_21-10-17

Workaround

  • As this could probably seen as an issue with changing a package ID after publishing, the workaround would be to simply not do that.

Analysis

  • It seems NuGet is officially treating package IDs case-insensitive, though this seems to be very rarely relevant / used.
  • Duality could work around this internally by using case-insensitive package ID comparison.
  • Probably low priority for now.
@ilexp ilexp added Bug It's broken and should be fixed Editor Area: Duality editor or support libraries Nice2Have Beneficial, but only very slightly so labels Dec 9, 2018
@ilexp ilexp added this to the General milestone Dec 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug It's broken and should be fixed Editor Area: Duality editor or support libraries Nice2Have Beneficial, but only very slightly so
Development

No branches or pull requests

1 participant