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

nuget.exe update drops the assembly strong name and Private attribute. #1778

Closed
emgarten opened this issue Nov 25, 2015 · 3 comments
Closed
Assignees
Labels
Functionality:Update The update package feature/command/experience Priority:3 Issues under consideration. With enough upvotes, will be reconsidered to be added to the backlog. Type:Bug
Milestone

Comments

@emgarten
Copy link
Member

nuget.exe update drops the assembly strong name and <Private> attribute.

This is not a regression from 2.8.6.

Steps

  1. Create a new C# console app
  2. Install-Package newtonsoft.json -version 6.0.4
  3. Save and close the solution
  4. nuget.3.3.0.exe update .\packages.config -Safe

From VS:

    <Reference Include="Newtonsoft.Json, Version=6.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed, processorArchitecture=MSIL">
      <HintPath>..\packages\Newtonsoft.Json.6.0.4\lib\net45\Newtonsoft.Json.dll</HintPath>
      <Private>True</Private>
    </Reference>

From nuget.exe:

    <Reference Include="Newtonsoft.Json">
      <HintPath>..\packages\newtonsoft.json.6.0.8\lib\net45\Newtonsoft.Json.dll</HintPath>
    </Reference>
@yishaigalatzer yishaigalatzer added Priority:3 Issues under consideration. With enough upvotes, will be reconsidered to be added to the backlog. Type:Bug labels Dec 31, 2015
@yishaigalatzer yishaigalatzer added this to the 3.4 milestone Dec 31, 2015
@yishaigalatzer yishaigalatzer modified the milestones: 3.4 Beta, 3.4 RTM Feb 25, 2016
@yishaigalatzer yishaigalatzer modified the milestones: 3.4 RTM, 3.4 RTM - Triage, 3.5 Beta Mar 11, 2016
@yishaigalatzer yishaigalatzer added the Functionality:Update The update package feature/command/experience label Mar 11, 2016
@yishaigalatzer yishaigalatzer assigned johnataylor and unassigned alpaix Mar 11, 2016
@yishaigalatzer yishaigalatzer modified the milestones: 3.5 Beta, 3.5 RC Apr 8, 2016
@jainaashish
Copy link
Contributor

NuGet/NuGet.Client#600

@zhili1208 zhili1208 added this to the 3.5 Beta2 milestone Jun 22, 2016
@zhili1208 zhili1208 removed this from the 3.6 Beta milestone Jun 22, 2016
@mdkf
Copy link

mdkf commented Sep 8, 2016

Just tested 3.5.0-RC1 command line client, instead of dropping the private tag it changed it from False to True.

@ferdaarikan
Copy link

Is the commit always adding the fully qualified name? The bug report was complaining about the strong name being removed. Seems like fix is always adding the strong name. We have some use cases that strong name is intentionally excluded from the reference entry. It seems like this fix is breaking the functionality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Functionality:Update The update package feature/command/experience Priority:3 Issues under consideration. With enough upvotes, will be reconsidered to be added to the backlog. Type:Bug
Projects
None yet
Development

No branches or pull requests

8 participants