{"action":"created","issue":{"url":"https://api.github.com/repos/OmniSharp/omnisharp-roslyn/issues/485","repository_url":"https://api.github.com/repos/OmniSharp/omnisharp-roslyn","labels_url":"https://api.github.com/repos/OmniSharp/omnisharp-roslyn/issues/485/labels{/name}","comments_url":"https://api.github.com/repos/OmniSharp/omnisharp-roslyn/issues/485/comments","events_url":"https://api.github.com/repos/OmniSharp/omnisharp-roslyn/issues/485/events","html_url":"https://github.com/OmniSharp/omnisharp-roslyn/pull/485","id":145197709,"number":485,"title":"Updated to new frameworks and CLI + extra sugaring in build script","user":{"login":"hal-ler","id":9846496,"avatar_url":"https://avatars.githubusercontent.com/u/9846496?v=3","gravatar_id":"","url":"https://api.github.com/users/hal-ler","html_url":"https://github.com/hal-ler","followers_url":"https://api.github.com/users/hal-ler/followers","following_url":"https://api.github.com/users/hal-ler/following{/other_user}","gists_url":"https://api.github.com/users/hal-ler/gists{/gist_id}","starred_url":"https://api.github.com/users/hal-ler/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hal-ler/subscriptions","organizations_url":"https://api.github.com/users/hal-ler/orgs","repos_url":"https://api.github.com/users/hal-ler/repos","events_url":"https://api.github.com/users/hal-ler/events{/privacy}","received_events_url":"https://api.github.com/users/hal-ler/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-04-01T14:38:39Z","updated_at":"2016-04-01T18:27:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/OmniSharp/omnisharp-roslyn/pulls/485","html_url":"https://github.com/OmniSharp/omnisharp-roslyn/pull/485","diff_url":"https://github.com/OmniSharp/omnisharp-roslyn/pull/485.diff","patch_url":"https://github.com/OmniSharp/omnisharp-roslyn/pull/485.patch"},"body":"Updated to the latest CLI and new framework names/packages.\r\n\r\nUpdating to the most recent CLI requires updating the version of `\"Microsoft.Extensions.DependencyModel\"`, which brings with itself dependencies on a newer NETStandard library. Since changes were required to the `project.json`, I decided to also update the framework monikers.\r\n\r\nChanges:\r\n\r\n- dnx451->net451\r\n\r\n- dnxcore50->netcoreapp1.0 for OmniSharp and tests and appropriate netstandard for the others\r\n\r\n- netstandard projects depend on `NETStandard.Library` (and other libraries if needed)\r\n\r\n- netcoreapp1.0 projects depend on `NETStandard.Library` and `Microsoft.DotNet.ProjectModel` (and other libraries if needed)\r\n    - NETCore.App brings extra unneeded dependencies including conflicting CodeAnalysis\r\n\r\n- Changed `NETStandard.Library`, `Microsoft.DotNet.*`, `Microsoft.Extensions.DependencyModel` , `System.*` dependency versions to use wild-cards for now (see below)\r\n\r\n- Fixed issues rising from new libraries\r\n\r\n- Tried to eliminate some of the unnecessary dependencies in the `project.json` files\r\n\r\n- Clean-up of `build.cake`\r\n    - I also removed Cake addins as they are third-party components targeting .NET 4.5, thus might not be available on a .NET Core version of Cake\r\n\r\n- Updated `.tools` path to match the recent change in moving `cake-bootstrap.*` into `scripts`\r\n\r\n- New utility target in `build.cake` named `SetPackageVersions`\r\n\r\n\r\nThings to fix long-term:\r\n- Eliminate imports for netstandard, especially dnxcore50\r\n\r\n     - Newtonsoft.Json only targets dnxcore50, this should change as soon as rc2 is released\r\n\r\n     - `Microsoft.Extensions.DependencyModel` pulls in `NuGet.Packaging`, which pulls in `System.Security.Cryptography.Hashing.Algorithms`, which only targets `dnxcore50` @troydai do you know any plans about this one?\r\n\r\n**SetPackageVersions**\r\n\r\nIt uses a new file called `depversion.json` as a central dictionary to replace all the dependency versions from `project.json` files across the different folders with the one specified within the file. This can be used to quickly migrate to new package versions. Dependencies without entries in `depversion.json` are left alone.\r\n\r\nThis target is not executed as part of the build, but can be called separately, by using `-target=SetPackageVersions` with `build.{ps1|sh}`.\r\n\r\nFor now `depversion.json` contains the packages that I specified with wild-card in this PR. This allows the wildcards to be quickly replaced by any version you guys prefer. Long-term the other dependencies can also be added here.\r\n"},"comment":{"url":"https://api.github.com/repos/OmniSharp/omnisharp-roslyn/issues/comments/204503530","html_url":"https://github.com/OmniSharp/omnisharp-roslyn/pull/485#issuecomment-204503530","issue_url":"https://api.github.com/repos/OmniSharp/omnisharp-roslyn/issues/485","id":204503530,"user":{"login":"troydai","id":1329240,"avatar_url":"https://avatars.githubusercontent.com/u/1329240?v=3","gravatar_id":"","url":"https://api.github.com/users/troydai","html_url":"https://github.com/troydai","followers_url":"https://api.github.com/users/troydai/followers","following_url":"https://api.github.com/users/troydai/following{/other_user}","gists_url":"https://api.github.com/users/troydai/gists{/gist_id}","starred_url":"https://api.github.com/users/troydai/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/troydai/subscriptions","organizations_url":"https://api.github.com/users/troydai/orgs","repos_url":"https://api.github.com/users/troydai/repos","events_url":"https://api.github.com/users/troydai/events{/privacy}","received_events_url":"https://api.github.com/users/troydai/received_events","type":"User","site_admin":false},"created_at":"2016-04-01T18:27:39Z","updated_at":"2016-04-01T18:27:39Z","body":"I'll suggest not to combine multiple changes in one PR. We either upgrade all the versions and open another PR for __ SetPackageVersions__ work. Combine them together lead to confusions."}}