Skip to content

Conversation

@mhutch
Copy link
Contributor

@mhutch mhutch commented Aug 3, 2016

On xbuild the MSBuildExtensionsPath property is multi-valued. However, this only works in Imports. When used in a property, it has a single value - which is not where the F# targets are installed on Mac.

As 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.

Also, make the locations we check for F# consistent across all the targets (maybe it should be factored out).

@dnfclas
Copy link

dnfclas commented Aug 3, 2016

Hi @mhutch, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;

@mhutch
Copy link
Contributor Author

mhutch commented Aug 3, 2016

(see also dotnet/android#141)

@xamarin-release-manager
Copy link
Collaborator

Build failure

@spouliot
Copy link
Contributor

spouliot commented Aug 4, 2016

@jstedfast can you review and merge (this one and the C8 based one) if everything is fine ? thanks!

@VincentDondain
Copy link
Contributor

That looks good to me 👍

@spouliot spouliot merged commit 3e04819 into dotnet:master Aug 9, 2016
@spouliot
Copy link
Contributor

spouliot commented Aug 10, 2016

@mhutch we have failures now on xcode8 for watchOS

/Users/builder/jenkins/workspace/xamarin-macios-pr-builder/tests/fsharplibrary/fsharplibrary-watchos.fsproj: error : /Users/builder/jenkins/workspace/xamarin-macios-pr-builder/tests/fsharplibrary/fsharplibrary-watchos.fsproj: /Users/builder/jenkins/workspace/xamarin-macios-pr-builder/_ios-build/Library/Frameworks/Mono.framework/External/xbuild/Xamarin/WatchOS/Xamarin.WatchOS.FSharp.targets: Project file could not be imported, it was being imported by /Users/builder/jenkins/workspace/xamarin-macios-pr-builder/tests/fsharplibrary/fsharplibrary-watchos.fsproj: The required attribute "Project" in Import is empty

most project files (for newer platforms) are generated from the "basic" classic project - so it might be that some change needs to be reflected into the tool that convert the projects

@mhutch
Copy link
Contributor Author

mhutch commented Aug 10, 2016

@spouliot the xcode8 branch does not have this commit. Maybe this needs to be backported...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants