Skip to content
This repository was archived by the owner on Jan 13, 2025. It is now read-only.

Conversation

@jpobst
Copy link
Contributor

@jpobst jpobst commented Jun 14, 2022

Author's note: I know very little about Cake or this repository's CI, so please review thoroughly! 😁

Changes to fix the currently broken XC CI:

  • Updating Cake had a breaking change that no longer allows empty CLI arguments to be passed with --foo="". It is now required to omit the equals sign.

  • Errors when trying to run build-manifest.cake fixed by removing methods 🤷‍♂️:

C:/code/XamarinComponents/.ci/build-manifest.cake(421,6): error CS0111: Type 'Submission#0' already defines a member called 'IsRunningOnMacOs' with the same parameter types
C:/code/XamarinComponents/.ci/build-manifest.cake(425,6): error CS0111: Type 'Submission#0' already defines a member called 'IsRunningOnLinux' with the same parameter types
  • Error when trying to run build-manifest.cake fixed by updating Cake.XCode:
C:/code/XamarinComponents/.ci/build-manifest.cake(234,4): error CS0103: The name 'CocoaPodRepoUpdate' does not exist in the current context
  • Revert requested XCode to 13.2.1. We are building on macos-11, which does not support XCode 13.3.1.

Note this PR will show as failed because CI is set up to use the .ci files from main. Here is a build that actually uses the files from the PR: https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=6282119.

@jpobst jpobst force-pushed the fix-build branch 5 times, most recently from 3784089 to 75542fa Compare June 14, 2022 16:31
@jpobst jpobst changed the title [ci] Revert cake version bump to fix CI. [ci] Changes to fix CI. Jun 14, 2022
@jpobst jpobst marked this pull request as ready for review June 14, 2022 16:56
@jpobst jpobst merged commit 9a5806c into main Jun 16, 2022
@jpobst jpobst deleted the fix-build branch June 16, 2022 14:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants