Changes to build in VSTS - #2362
Closed
etherealiska wants to merge 15 commits into
Closed
Conversation
|
@joewstroman, It will cover your contributions to all .NET Foundation-managed open source projects. |
Contributor
|
Build failure |
1 similar comment
Contributor
|
Build failure |
Contributor
|
@joewstroman we have limited bots capacity. Please make sure it builds locally before doing PR. Thanks! |
Contributor
|
Build failure |
Contributor
Author
|
I'm not sure why this failed. It hangs for 20 minutes while running tests, but none of these changes touch any xamarin.mac or xamarin.ios code. I think xam-mac-mini-1 might be hosed @spouliot . It's failing more often than other bots. |
Contributor
|
Build failure |
etherealiska
force-pushed
the
vsts-sys-deps
branch
from
July 26, 2017 21:05
3f2c963 to
5e53068
Compare
Contributor
|
Build success |
|
@joewstroman, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request. |
Member
|
The changes to system-dependencies.sh are fine, but I don't understand why you can't just clone release-scripts manually executing "git clone ..." in VSTS? Copied files always end up stale one day. |
Contributor
Author
|
I saw that the Jenkins PR's use that script, so I assumed that's the way it should continue happening. I can clone release-scripts, but I believe we're moving away from that repo for VSTS? @duncanmak ? |
rolfbjarne
added a commit
to rolfbjarne/macios
that referenced
this pull request
Jan 14, 2021
New commits in xamarin/maccore: * xamarin/maccore@2d9f8a9685 Bump to xamarin/xamarin-analysis@95245313 (dotnet#2362) * xamarin/maccore@428964bf32 [Device tests] Remove all the code related with vsts pipelines. (dotnet#2344) * xamarin/maccore@3875d125da Ignore vscode config files. (dotnet#2343) * xamarin/maccore@55ca0540ef [Actions] Add a rebase action. (dotnet#2341) * xamarin/maccore@d0bad0f87a [Actions] Add the backport action. (dotnet#2340) Diff: https://github.com/xamarin/maccore/compare/5133d2cea1c52fb783f64ab2ba97c760cf33f5f4..2d9f8a96856fbd40f13a8e57df9c3d23e69602fb
Merged
rolfbjarne
added a commit
that referenced
this pull request
Jan 20, 2021
New commits in xamarin/maccore: * xamarin/maccore@55a700bb5b [build] Add /restore to Xamarin.Analysis build (#2366) * xamarin/maccore@2d9f8a9685 Bump to xamarin/xamarin-analysis@95245313 (#2362) * xamarin/maccore@428964bf32 [Device tests] Remove all the code related with vsts pipelines. (#2344) * xamarin/maccore@3875d125da Ignore vscode config files. (#2343) * xamarin/maccore@55ca0540ef [Actions] Add a rebase action. (#2341) * xamarin/maccore@d0bad0f87a [Actions] Add the backport action. (#2340) Diff: https://github.com/xamarin/maccore/compare/5133d2cea1c52fb783f64ab2ba97c760cf33f5f4..55a700bb5ba3c092f511075d479d82ecb10fa5fe Co-authored-by: Peter Collins <pecolli@microsoft.com>
spouliot
pushed a commit
to spouliot/xamarin-macios
that referenced
this pull request
Feb 4, 2021
New commits in xamarin/maccore: * xamarin/maccore@de616ca53e [mlaunch] Fix dependencies for Xcode 12.5 beta 1 (dotnet#2387) * xamarin/maccore@e93375583c [Actions] Fix rebase trigger. * xamarin/maccore@0dfeb85ff6 [CI][VSTS] Add information when the external tool fails. (dotnet#2377) * xamarin/maccore@87e165fdbb Remove Vincent as a code owner 😞 (dotnet#2368) * xamarin/maccore@55a700bb5b [build] Add /restore to Xamarin.Analysis build (dotnet#2366) * xamarin/maccore@2d9f8a9685 Bump to xamarin/xamarin-analysis@95245313 (dotnet#2362) Diff: https://github.com/xamarin/maccore/compare/428964bf32f06862eb4209129c6fb5b882c3b9be..de616ca53e22ebea66216374a15b8caf2ca77e3c
spouliot
added a commit
that referenced
this pull request
Feb 6, 2021
This version of Xcode requires macOS 11.0 or later. New commits in xamarin/maccore: * xamarin/maccore@de616ca53e [mlaunch] Fix dependencies for Xcode 12.5 beta 1 (#2387) * xamarin/maccore@e93375583c [Actions] Fix rebase trigger. * xamarin/maccore@0dfeb85ff6 [CI][VSTS] Add information when the external tool fails. (#2377) * xamarin/maccore@87e165fdbb Remove Vincent as a code owner 😞 (#2368) * xamarin/maccore@55a700bb5b [build] Add /restore to Xamarin.Analysis build (#2366) * xamarin/maccore@2d9f8a9685 Bump to xamarin/xamarin-analysis@95245313 (#2362) Diff: https://github.com/xamarin/maccore/compare/428964bf32f06862eb4209129c6fb5b882c3b9be..de616ca53e22ebea66216374a15b8caf2ca77e3c * Bump system mono to the latest available 2020-02 package. (#10402) This makes xamarin-macios build on Apple Silicon, and also seems to get an updated csc that fixes a problem with nullability warnings/errors. * [tests] Fix mmp-regression build ``` 10:59:46.5383210 Making all in /Users/builder/azdo/_work/1/s/xamarin-macios/tests/mmp-regression 10:59:46.5392610 TargetFrameworkFallbackSearchPaths=/Users/builder/azdo/_work/1/s/xamarin-macios/_mac-build/Library/Frameworks/Mono.framework/External/xbuild-frameworks MSBuildExtensionsPathFallbackPathsOverride=/Users/builder/azdo/_work/1/s/xamarin-macios/_mac-build/Library/Frameworks/Mono.framework/External/xbuild make all 10:59:46.6208220 /Applications/Xcode_12.5.0-beta.app/Contents/Developer/usr/bin/make clean 10:59:46.6956100 /Applications/Xcode_12.5.0-beta.app/Contents/Developer/usr/bin/make build 10:59:46.7422460 GEN dylib/libTest.dylib 10:59:46.8094820 ld: library not found for -lSystem 10:59:46.8097590 clang: error: linker command failed with exit code 1 (use -v to see invocation) 10:59:46.8110310 make[2]: *** [dylib/libTest.dylib] Error 1 10:59:46.8112570 make[1]: *** [run] Error 2 10:59:47.0030120 Process make exited with 2 ``` This happens on BigSur as it does not find the default location anymore... * [tests] Do not test GKLeaderboardViewController on BigSur The issue is the OS version, not the Xcode version. Update check to skip that test on macOS 11+ * [tests] Fix MSBuild tests / integration The test cannot work on BigSur since while you can `dlopen` the library you can open `fopen` it as a file (and can't embed it for the test). ``` 12:31:23.9407820 CSC : error CS1566: Error reading resource 'libz.dylib' -- 'Could not find file "/usr/lib/libz.dylib"' [/Users/builder/azdo/_work/1/s/xamarin-macios/tests/msbuild/Xamarin.MacDev.Tests/bin/Debug/net472/tmp-test-dir/msbuild-tests201/XM45Binding.csproj] 12:31:23.9407900 Done Building Project "/Users/builder/azdo/_work/1/s/xamarin-macios/tests/msbuild/Xamarin.MacDev.Tests/bin/Debug/net472/tmp-test-dir/msbuild-tests201/XM45Binding.csproj" (default targets) -- FAILED. ``` * [tests] Fix MTouch tests failures * `nm` changed it's output and test was updated * `clang` linking has a much higher limit for it's command-line length At least it does not seem to be OS specific since both 10.15 and 11.0 returns the same maximum ``` $ getconf ARG_MAX 1048576 ```
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
VSTS doesn't really support multiple checkouts, so I thought it would be easiest to add the package targets and other build scripts here (instead of release-scripts). Also VSTS doesn't allow the use of sudo, so I copied the system-dependencies file. It seemed like less of a hassle.