{"action":"created","issue":{"url":"https://api.github.com/repos/xamarin/xamarin-macios/issues/541","repository_url":"https://api.github.com/repos/xamarin/xamarin-macios","labels_url":"https://api.github.com/repos/xamarin/xamarin-macios/issues/541/labels{/name}","comments_url":"https://api.github.com/repos/xamarin/xamarin-macios/issues/541/comments","events_url":"https://api.github.com/repos/xamarin/xamarin-macios/issues/541/events","html_url":"https://github.com/xamarin/xamarin-macios/pull/541","id":169256974,"number":541,"title":"Fix F# targets on xbuild 12.0 & 14.0","user":{"login":"mhutch","id":183285,"avatar_url":"https://avatars.githubusercontent.com/u/183285?v=3","gravatar_id":"","url":"https://api.github.com/users/mhutch","html_url":"https://github.com/mhutch","followers_url":"https://api.github.com/users/mhutch/followers","following_url":"https://api.github.com/users/mhutch/following{/other_user}","gists_url":"https://api.github.com/users/mhutch/gists{/gist_id}","starred_url":"https://api.github.com/users/mhutch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mhutch/subscriptions","organizations_url":"https://api.github.com/users/mhutch/orgs","repos_url":"https://api.github.com/users/mhutch/repos","events_url":"https://api.github.com/users/mhutch/events{/privacy}","received_events_url":"https://api.github.com/users/mhutch/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/xamarin/xamarin-macios/labels/cla-already-signed","name":"cla-already-signed","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"jstedfast","id":338984,"avatar_url":"https://avatars.githubusercontent.com/u/338984?v=3","gravatar_id":"","url":"https://api.github.com/users/jstedfast","html_url":"https://github.com/jstedfast","followers_url":"https://api.github.com/users/jstedfast/followers","following_url":"https://api.github.com/users/jstedfast/following{/other_user}","gists_url":"https://api.github.com/users/jstedfast/gists{/gist_id}","starred_url":"https://api.github.com/users/jstedfast/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jstedfast/subscriptions","organizations_url":"https://api.github.com/users/jstedfast/orgs","repos_url":"https://api.github.com/users/jstedfast/repos","events_url":"https://api.github.com/users/jstedfast/events{/privacy}","received_events_url":"https://api.github.com/users/jstedfast/received_events","type":"User","site_admin":false},"assignees":[{"login":"jstedfast","id":338984,"avatar_url":"https://avatars.githubusercontent.com/u/338984?v=3","gravatar_id":"","url":"https://api.github.com/users/jstedfast","html_url":"https://github.com/jstedfast","followers_url":"https://api.github.com/users/jstedfast/followers","following_url":"https://api.github.com/users/jstedfast/following{/other_user}","gists_url":"https://api.github.com/users/jstedfast/gists{/gist_id}","starred_url":"https://api.github.com/users/jstedfast/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jstedfast/subscriptions","organizations_url":"https://api.github.com/users/jstedfast/orgs","repos_url":"https://api.github.com/users/jstedfast/repos","events_url":"https://api.github.com/users/jstedfast/events{/privacy}","received_events_url":"https://api.github.com/users/jstedfast/received_events","type":"User","site_admin":false}],"milestone":null,"comments":5,"created_at":"2016-08-03T23:21:00Z","updated_at":"2016-08-09T15:18:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/xamarin/xamarin-macios/pulls/541","html_url":"https://github.com/xamarin/xamarin-macios/pull/541","diff_url":"https://github.com/xamarin/xamarin-macios/pull/541.diff","patch_url":"https://github.com/xamarin/xamarin-macios/pull/541.patch"},"body":"On xbuild the `MSBuildExtensionsPath` property is multi-valued. However, this only works in `Import`s. When used in a property, it has a single value - which is not where the F# targets are installed on Mac. \r\n\r\nAs as result, the Xamarin targets were falling back to checking `MSBuildBinPath`, however the F# targets were only installed into the bin path of the 4.0 toolset, and so this failed with the 12.0 and 14.0 toolsets.\r\n\r\nAlso, make the locations we check for F# consistent across all the targets (maybe it should be factored out)."},"comment":{"url":"https://api.github.com/repos/xamarin/xamarin-macios/issues/comments/238587188","html_url":"https://github.com/xamarin/xamarin-macios/pull/541#issuecomment-238587188","issue_url":"https://api.github.com/repos/xamarin/xamarin-macios/issues/541","id":238587188,"user":{"login":"VincentDondain","id":7839202,"avatar_url":"https://avatars.githubusercontent.com/u/7839202?v=3","gravatar_id":"","url":"https://api.github.com/users/VincentDondain","html_url":"https://github.com/VincentDondain","followers_url":"https://api.github.com/users/VincentDondain/followers","following_url":"https://api.github.com/users/VincentDondain/following{/other_user}","gists_url":"https://api.github.com/users/VincentDondain/gists{/gist_id}","starred_url":"https://api.github.com/users/VincentDondain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/VincentDondain/subscriptions","organizations_url":"https://api.github.com/users/VincentDondain/orgs","repos_url":"https://api.github.com/users/VincentDondain/repos","events_url":"https://api.github.com/users/VincentDondain/events{/privacy}","received_events_url":"https://api.github.com/users/VincentDondain/received_events","type":"User","site_admin":false},"created_at":"2016-08-09T15:18:50Z","updated_at":"2016-08-09T15:18:50Z","body":"That looks good to me 👍 "}}
{"actor":{"display_login":"VincentDondain"}}