Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Revert "[ci] Switch to macOS VM with Flutter pre-installed" #1864

Closed
wants to merge 1 commit into from

Conversation

amirh
Copy link
Contributor

@amirh amirh commented Jul 16, 2019

Reverts #1313

We've been started to get the following error on the Cirrus tasks:

./script/incremental_build.sh build-examples --ipa
fatal: No such ref: 'FETCH_HEAD'
fatal: Not a valid object name FETCH_HEAD

I suspect the reverted change is where it regressed.

@fkorotkov
Copy link
Contributor

@amirh do you have a link to the build where you saw it?

@amirh
Copy link
Contributor Author

amirh commented Jul 16, 2019

It happens on any macOS task since this change landed, I think what happens is that ./script/incremental_build.sh expects the current directory to be one where the plugins repo is cloned but it isn't(though let's first see if we're not getting the error with the revert).

e.g see the log for build here: https://cirrus-ci.com/task/4977285139005440

@fkorotkov
Copy link
Contributor

I think it's because git fetch origin master got removed. My bad! See #1865

@collinjackson
Copy link
Contributor

Closing this in favor of #1865

collinjackson pushed a commit that referenced this pull request Jul 17, 2019
This also fetches master that will populate `FETCH_HEAD`

See #1864
mithun-mondal pushed a commit to bKash-developer/archived_plugins that referenced this pull request Aug 6, 2019
This also fetches master that will populate `FETCH_HEAD`

See flutter#1864
collinjackson pushed a commit to firebase/flutterfire that referenced this pull request Aug 14, 2019
This also fetches master that will populate `FETCH_HEAD`

See flutter/plugins#1864
@bparrishMines bparrishMines deleted the revert-1313-patch-6 branch August 22, 2019 20:59
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
…#1864)

* implement missing auth functionality for web

* add changelog

* bump pubspec version

* add missing method to changelog

* fix interface
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants