Remove transitive dependencies already included by other projects

[pmiossec]

and found by snitch tool https://github.com/spectresystems/snitch

snitch GitExtensions.sln

(projects with only dependency global Microsoft.VisualStudio.Threading have been removed)

┌─────────────────────────────────────────────────────────────────────────────────┐
│ Packages that can be removed from GitUI: │
│ ┌──────────────────────────────────────────────────────┬──────────────────────┐ │
│ │ Package │ Referenced by │ │
│ ├──────────────────────────────────────────────────────┼──────────────────────┤ │
│ │ System.IO.Abstractions │ GitCommands │ │
│ │ Microsoft.VisualStudio.Threading │ GitExtUtils │ │
│ └──────────────────────────────────────────────────────┴──────────────────────┘ │
│ │
│ Packages that can be removed from GitExtensions.Plugins.AutoCompileSubmodules: │
│ ┌───────────────────────────────────────────────┬─────────────────────────────┐ │
│ │ Package │ Referenced by │ │
│ ├───────────────────────────────────────────────┼─────────────────────────────┤ │
│ │ Microsoft.VisualStudio.Composition │ GitUIPluginInterfaces │ │
│ │ System.Reactive.Interfaces │ GitUIPluginInterfaces │ │
│ │ Microsoft.VisualStudio.Threading │ GitExtUtils │ │
│ └───────────────────────────────────────────────┴─────────────────────────────┘ │
│ │
│ Packages that can be removed from TeamCityIntegration: │
│ ┌──────────────────────────────────────────────────────┬──────────────────────┐ │
│ │ Package │ Referenced by │ │
│ ├──────────────────────────────────────────────────────┼──────────────────────┤ │
│ │ System.Reactive.Linq │ GitCommands │ │
│ │ Microsoft.VisualStudio.Threading │ GitExtUtils │ │
│ └──────────────────────────────────────────────────────┴──────────────────────┘ │
│ │
│ Packages that can be removed from GitExtensions.Plugins.Bitbucket: │
│ ┌──────────────────────────────────────────────────────┬──────────────────────┐ │
│ │ Package │ Referenced by │ │
│ ├──────────────────────────────────────────────────────┼──────────────────────┤ │
│ │ Newtonsoft.Json │ GitCommands │ │
│ │ Microsoft.VisualStudio.Threading │ GitExtUtils │ │
│ └──────────────────────────────────────────────────────┴──────────────────────┘ │
│ │
│ Packages that can be removed from JenkinsIntegration: │
│ ┌──────────────────────────────────────────────────────┬──────────────────────┐ │
│ │ Package │ Referenced by │ │
│ ├──────────────────────────────────────────────────────┼──────────────────────┤ │
│ │ Newtonsoft.Json │ GitCommands │ │
│ │ System.Reactive.Linq │ GitCommands │ │
│ │ Microsoft.VisualStudio.Threading │ GitExtUtils │ │
│ └──────────────────────────────────────────────────────┴──────────────────────┘ │
│ │
│ Packages that can be removed from AppVeyorIntegration: │
│ ┌──────────────────────────────────────────────────────┬──────────────────────┐ │
│ │ Package │ Referenced by │ │
│ ├──────────────────────────────────────────────────────┼──────────────────────┤ │
│ │ Newtonsoft.Json │ GitCommands │ │
│ │ System.Reactive.Linq │ GitCommands │ │
│ │ Microsoft.VisualStudio.Threading │ GitExtUtils │ │
│ └──────────────────────────────────────────────────────┴──────────────────────┘ │
│ │
│ Packages that can be removed from GitCommands.Tests: │
│ ┌──────────────────────────────────────────────────────┬──────────────────────┐ │
│ │ Package │ Referenced by │ │
│ ├──────────────────────────────────────────────────────┼──────────────────────┤ │
│ │ Newtonsoft.Json │ GitCommands │ │
│ │ Microsoft.VisualStudio.Threading │ GitExtUtils │ │
│ └──────────────────────────────────────────────────────┴──────────────────────┘ │
│ │
│ Packages that can be removed from GitUI.Tests: │
│ ┌─────────────────────────────────────────────────────┬───────────────────────┐ │
│ │ Package │ Referenced by │ │
│ ├─────────────────────────────────────────────────────┼───────────────────────┤ │
│ │ System.IO.Abstractions │ GitCommands │ │
│ │ Microsoft.VisualStudio.Threading │ ConEmuWinForms │ │
│ └─────────────────────────────────────────────────────┴───────────────────────┘ │
│ │
│ Packages that can be removed from AzureDevOpsIntegration: │
│ ┌──────────────────────────────────────────────────────┬──────────────────────┐ │
│ │ Package │ Referenced by │ │
│ ├──────────────────────────────────────────────────────┼──────────────────────┤ │
│ │ Newtonsoft.Json │ GitCommands │ │
│ │ System.Reactive.Linq │ GitCommands │ │
│ │ Microsoft.VisualStudio.Threading │ GitExtUtils │ │
│ └──────────────────────────────────────────────────────┴──────────────────────┘ │
└─────────────────────────────────────────────────────────────────────────────────┘

Test methodology

• Nothing really changed....

Merge strategy

I agree that the maintainer squash merge this PR (if the commit message is clear).

---

✒️ I contribute this code under The Developer Certificate of Origin.

[pmiossec]

Notes:

• The displayed table render better in GitExtensions ;)
• A little cleaning but no huge added value as the version is managed globally (a we could not end up with 2 projects using different versions of a package). If you consider that it doesn't worth it, feel free to close the PR...

[gerhardol]

+1

[RussKie]

You'd find that below we have the following code:

  <ItemGroup>
    <PluginAssembly Include="$([System.IO.Path]::Combine('$(TargetDir)', 'Newtonsoft.Json.dll'))" />
  </ItemGroup>

If we remove this line this entry becomes very difficult to reason about... I don't have a good suggestion atm, just making a note.

I wonder if we can get away from using Newtonsoft.Json.dll at all and use System.Text.Json API instead. But this is a completely different story...