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

Incrementally bump mono #3106

Closed
wants to merge 5 commits into from

Conversation

spouliot
Copy link
Contributor

Commit list for mono/mono:

Diff: mono/mono@c5cd0f1...b3b0613

https://bugzilla.xamarin.com/show_bug.cgi?id=60770

emaf and others added 2 commits December 14, 2017 20:47
MoveTaskBase inherits from Microsoft.Build.Tasks.Move, and Mono has a different implementation of it, so when building from a Mac we need to include XBuildMoveTaskBase.cs instead. The previous condition does no apply any more, because we're now using MSBuild to build, but the Move task implementation didn't change so it doesn't matter if we're using Xbuild or MSBuild.

This issue is preventing us (XVS) from merging features to our master branch
Commit list for mono/mono:

* mono/mono@b3b0613ad38 Bump msbuild to bring in fix for #60770 (xamarin#6107)
* mono/mono@ddeba6e1bab [interp] fix using conv.u with string
* mono/mono@0360f420fe3 Bump API snapshot submodule
* mono/mono@2f18e7dd23c Bump cecil & linker to match master.
* mono/mono@0f53cb275c4 [interp] allow unsigned i8 in pinvoke signature

Diff: mono/mono@c5cd0f1...b3b0613

https://bugzilla.xamarin.com/show_bug.cgi?id=60770
@spouliot spouliot added do-not-merge Do not merge this pull request run-all-tests Run all our tests. labels Dec 15, 2017
@monojenkins
Copy link
Collaborator

Build failure

@monojenkins
Copy link
Collaborator

Build failure

@monojenkins
Copy link
Collaborator

Build failure

@monojenkins
Copy link
Collaborator

Build failure

@spouliot
Copy link
Contributor Author

no doubt, the issues starts with the bump from 2f18e7

@spouliot
Copy link
Contributor Author

binary search inside that bump won't be done by this PR :)

@spouliot spouliot closed this Dec 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge Do not merge this pull request run-all-tests Run all our tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants