Skip to content

Mono 2017-02 integration - #1872

Merged
spouliot merged 77 commits into
masterfrom
mono-2017-02
Mar 29, 2017
Merged

Mono 2017-02 integration#1872
spouliot merged 77 commits into
masterfrom
mono-2017-02

Conversation

@marek-safar

@marek-safar marek-safar commented Mar 14, 2017

Copy link
Copy Markdown
Contributor

No description provided.

@marek-safar
Bump to momo master
9d84f1c
@marek-safar
[build] Profiles cleanup
6a20abc
@marek-safar
Replace mdbrebase tool with pathmap and fix broken debug symbols mapp…
1f606b5 
…ing in public builds
@marek-safar
Remove 4.x profile dependency
1e77e19
@marek-safar
Bump mono and linker submodule
1d3ad5e
@marek-safar
Add netstandard to the build
1020340
@marek-safar
[build] Register linker for submodule checks
ddb377b
@marek-safar
[build] Add .NET Core open source key
2c9c0ee
@marek-safar
Bump mono
398935d
@marek-safar
[build] Update debug symbols to pdb
16ba9f1
@marek-safar
[msbuild] Remove smcs compiler overrides
e47eaff
@marek-safar
Update after another Cecil breaking change
01f0e67
@marek-safar
Fixes mmp execution during build to use correct Mono.Cecil. For some …
4dc2abc 
…reason setting MONO_PATH does not work
@marek-safar
[linker] Update list of Facade assemblies
2ca93a9
@marek-safar
[mtouch] Copy pdb debug symbols when available
c51ba19
@marek-safar
[msbuild] Update test to reflect Mono.Dynamic.Interpreter.dll removal
741fc32
@marek-safar
[tuner] Update System.Core rule
9a60036
@marek-safar
[msbuild] Update few test expected debug symbols
f2d61b6
@marek-safar
Fixes missing Mono.Security dependency
01f3133
@marek-safar
Switch to Mono 2017-02 branch
aceedd8
@marek-safar
Adds pdb handling to mtouch
258cbdd
@marek-safar
Updates mtouch.csproj
d0ea158
@marek-safar
Bump linker
24f9d3a
@marek-safar
Wire up configuration to MonoTouchSweepStep
60bf37e
@marek-safar
Mono branch name update
fbc3f2e
@marek-safar
Remove mdbrebase which is no longer needed (compiler does it now)
6906d89
@marek-safar
Update monomac tuner
0bad977
@marek-safar
Revert "[linker] Inline BinaryCompatibility.TargetsAtLeast_Desktop_V4…
0913aec 
…_5[_1] (#1518)"

This reverts commit e72b654.

because it's no longer needed with latest mono
@marek-safar
Bump maccore for mdoc path fix
50f4564
@marek-safar
Add missing reference
46ee015
@spouliot

spouliot commented Mar 24, 2017

Copy link
Copy Markdown
Contributor

https://bugzilla.xamarin.com/show_bug.cgi?id=53481 is presently blocking QA to complete testing

@monojenkins

monojenkins commented Mar 24, 2017

Copy link
Copy Markdown
Contributor

Build failure

@spouliot

spouliot commented Mar 24, 2017

Copy link
Copy Markdown
Contributor

@kzu @josegallardo @emaf I have not heard any comment / go ahead on the msbuild parts. A quick thumb up would be appreciated. thanks!

@monojenkins

monojenkins commented Mar 24, 2017

Copy link
Copy Markdown
Contributor

Build failure

@monojenkins

monojenkins commented Mar 26, 2017

Copy link
Copy Markdown
Contributor

Build failure

@rolfbjarne

rolfbjarne commented Mar 27, 2017

Copy link
Copy Markdown
Member

build

@monojenkins

monojenkins commented Mar 27, 2017

Copy link
Copy Markdown
Contributor

Build failure

@rolfbjarne rolfbjarne mentioned this pull request Mar 27, 2017
Merged
emaf
emaf approved these changes Mar 27, 2017
View reviewed changes

@emaf emaf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The msbuild changes look good to me

kzu
kzu approved these changes Mar 27, 2017
View reviewed changes

@kzu kzu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rolfbjarne @spouliot the PR looks fine. We'll just need to delete the PDB2MDB stuff on our side. I assume this will also be the case for XA, @jonpryor?

This basically means we'll likely be able to remove altogether the windows-only targets that XVS installs, I hope.

We'll adapt our side once this is merged.

Thanks and sorry for the delay!

@monojenkins

monojenkins commented Mar 28, 2017

Copy link
Copy Markdown
Contributor

Build failure

@monojenkins

monojenkins commented Mar 28, 2017

Copy link
Copy Markdown
Contributor

Build failure

1 similar comment
@monojenkins

monojenkins commented Mar 28, 2017

Copy link
Copy Markdown
Contributor

Build failure

@monojenkins

monojenkins commented Mar 29, 2017

Copy link
Copy Markdown
Contributor

Build failure

@monojenkins

monojenkins commented Mar 29, 2017

Copy link
Copy Markdown
Contributor

Build failure

@monojenkins

monojenkins commented Mar 29, 2017

Copy link
Copy Markdown
Contributor

Build failure

spouliot
spouliot approved these changes Mar 29, 2017
View reviewed changes

@spouliot spouliot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA approved for merging.
Remaining issues to be solved afterward.

@spouliot
spouliot merged commit 743c28f into master Mar 29, 2017
@monojenkins

monojenkins commented Mar 29, 2017

Copy link
Copy Markdown
Contributor

Build failure

@mandel-macaque
mandel-macaque deleted the mono-2017-02 branch December 7, 2021 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rolfbjarne rolfbjarne rolfbjarne approved these changes
@emaf emaf emaf approved these changes
@VincentDondain VincentDondain Awaiting requested review from VincentDondain
@josegallardo josegallardo Awaiting requested review from josegallardo
+4 more reviewers
@kzu kzu kzu approved these changes
@spouliot spouliot spouliot approved these changes
@jstedfast jstedfast jstedfast approved these changes
@chamons chamons chamons approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@spouliot spouliot

Labels

enable-device-build Makes our build include device support (which we disable for simple PRs to speed them up) requires-approval-before-merge The pull request requires special approval before it can be merged requires-qa-before-merge The pull request requires QA to approve it before it can be merged run-all-tests Run all our tests.

Projects

None yet

Milestone

d15-2

Development

Successfully merging this pull request may close these issues.

10 participants

@marek-safar @monojenkins @spouliot @jstedfast @rolfbjarne @kzu @akoeplinger @emaf @chamons @dnfclas