Skip to content

Can't run flutter test on a Mac with prebuilt binaries #1707

Closed

Description

@eseidelGoogle

We just don't upload the binaries to google storage, but we could.

For now I'm disabling running flutter test tests on our new mac infra.

Activity

  1. eseidelGoogle commented on Feb 9, 2016

    @eseidelGoogle
    ContributorAuthor

    Disabling flutter test on the mac bots for now: https://codereview.chromium.org/1678283004/

  2. abarth commented on Feb 12, 2016

    @abarth
    Contributor

    You can run flutter test on a mac if you have an engine build. Re-titling.

  3. changed the title [-]Can't run flutter test on a Mac[/-] [+]Can't run flutter test on a Mac with prebuilt binaries[/+] on Feb 12, 2016
  4. eseidelGoogle commented on Feb 12, 2016

    @eseidelGoogle
    ContributorAuthor

    Just change this one line of code:
    https://chromium.googlesource.com/chromium/tools/build.git/+/master/scripts/slave/recipes/flutter/engine.py#141
    And we will suddenly have prebuilt binaries for the Mac. :) (uploaded to flutter_infra instead of mojo_infra) I suspect you and I will do that today in our infra handoff.

  5. modified the milestone: on Feb 16, 2016
  6. eseidelGoogle commented on Feb 19, 2016

    @eseidelGoogle
    ContributorAuthor

    I briefly looked at adding this just now. The (small) complexity is that SkyShell.app is a directory and the current UploadArtifacts logic expects files:
    https://chromium.googlesource.com/chromium/tools/build.git/+/master/scripts/slave/recipes/flutter/engine.py#48

    We can teach it to check if things are files or directories and call AddDirectory instead when necessary.

  7. eseidelGoogle commented on Feb 19, 2016

    @eseidelGoogle
    ContributorAuthor

    https://codereview.chromium.org/1713153002/ was my attempt, likely wrong though.

  8. eseidelGoogle commented on Feb 23, 2016

    @eseidelGoogle
    ContributorAuthor

    My impression is @chinmaygarde may have started on this too.

  9. modified the milestones: Next, on Mar 4, 2016
  10. added
    platform-macosBuilding on or for macOS specifically
    and removed
    platform-iosiOS applications specifically
    on Mar 24, 2016
  11. eseidelGoogle commented on Apr 12, 2016

    @eseidelGoogle
    ContributorAuthor

    Second attempt at adding SkyShell.app to the zip:
    https://codereview.chromium.org/1884703002

  12. chinmaygarde commented on May 19, 2016

    @chinmaygarde
    Contributor
  13. pylaligand commented on May 19, 2016

    @pylaligand
    Contributor

    Wow, I was just about to file a bug for this as I'm getting:

    Cannot find Flutter shell at /foo/bar/blah/src/third_party/flutter/bin/cache/artifacts/engine/darwin-x64/sky_shell
    

    Thanks!

  14. chinmaygarde commented on May 19, 2016

    @chinmaygarde
    Contributor

    @pylaligand You will have to wait a few minutes till #4028 lands.

  15. github-actions commented on Sep 6, 2021

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  16. locked as resolved and limited conversation to collaborators on Sep 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

c: new featureNothing broken; request for a new capabilityplatform-macosBuilding on or for macOS specifically

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions