-
Notifications
You must be signed in to change notification settings - Fork 511
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
[msbuild] Move from xbuild to msbuild #4111
Merged
Merged
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
c7b0514
[msbuild] Replace all invocations of xbuild with msbuild
radical 256035e
[msbuild] Update xbuild assembly references to msbuild ones
radical a5670f2
[msbuild] Port the tasks and tests to use new msbuild API
radical 18f7168
[tests] Fix DetectAppManifest_ExecutableProject_NoPList
radical 2805bea
[tests] Ensure that the correct msbuild.dll.config gets picked up
radical b883c4f
[tests] Don't use WriteLine in TestHelpers.Loggers as msbuild adds that
radical 9e53fd5
[tests] Xamarin.iOS.Tasks.TargetTests.RebuildExecutable_TouchLibraryDll
radical 6fc89ab
[tests] msbuild-mac Fix tests depending on msbuild specific strings
radical 26abe1b
[tests] Fix tests that depend on tool output
radical 8f0df14
[tests] Remove project path at the end of errors/warnings before comp…
radical 6278f10
[tests] Remove `bool useMSBuild` arg from ProjectTestHelpers
radical b578593
[tests] Remove TI.UnifiedTestConfig.DiagnosticMSBuild
radical 526f009
[tests] Fix Xamarin.iOS.Tasks.TargetTests.GenerateBundleName_Executab…
radical 16e512a
Revert "[msbuild] Add forward compatibility with symlinks support (#6…
radical 516aab9
Merge remote-tracking branch 'origin/master' into msbuild-port
rolfbjarne 19d9a64
[xharness] Don't add LogicalName beneath Import elements, it doesn't …
rolfbjarne afcad3b
[xharness] Set the right variable to point msbuild to our locally bui…
rolfbjarne 50e0554
[xharness] Process Import elements, but don't add LogicalName to them.
rolfbjarne 06d7410
Remove debug spew.
rolfbjarne 2e75028
[tests] Set XamarinMacFrameworkRoot as an environment variable when r…
rolfbjarne acad87b
Bump system mono to 5.16 to see if that gets an MSBuild with the fixe…
rolfbjarne 42645d5
[tests][msbuild] Seems like we need to copy System.Reflection.Metadat…
rolfbjarne 745c18c
Merge remote-tracking branch 'origin/master' into msbuild-port
rolfbjarne 4362842
[mmptests] Update according to MSBuild changes.
rolfbjarne 5fbfacc
[tests] Only restore packages.config, not the mtouch test project.
rolfbjarne File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Someone should do %s/SYSTEM_XBUILD/SYSTEM_MSBUILD/gc at some later point.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
um should I do it in this PR itself?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll do it as commit after all the changes here, that should keep the review of the patches easy and clean. And this will be just a safe cosmetic fix.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will open a new PR for that.