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

Conversation

@aam
Copy link
Member

@aam aam commented Oct 23, 2024

Copy gen_snapshots using python's shutil.copy, avoid links

Default implementation of copy does it via hardlink, which seems to be causing issues with Gatekeeper on mac.

This is reland of 6c4f05c with the fix for the case when target is a link to the source, that was the cause for revert 78f4ffa

BUG=flutter/flutter#154437

Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM stamp from a Japanese personal seal

@aam aam merged commit 4f28e05 into flutter:main Oct 23, 2024
@aam aam deleted the reland-copy-gen-snapshot branch October 23, 2024 17:26
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 23, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 23, 2024
flutter/engine@6b022df...19f9c5a

2024-10-23 skia-flutter-autoroll@skia.org Roll Skia from ad09ab099fe4 to 721bbd7c0340 (10 revisions) (flutter/engine#56063)
2024-10-23 aam@google.com Reland copy gen snapshot (flutter/engine#56061)
2024-10-23 skia-flutter-autoroll@skia.org Roll Dart SDK from 2a4b728f6a03 to 75c42f30af7a (1 revision) (flutter/engine#56060)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC codefu@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
M97Chahboun pushed a commit to M97Chahboun/flutter that referenced this pull request Oct 30, 2024
…7460)

flutter/engine@6b022df...19f9c5a

2024-10-23 skia-flutter-autoroll@skia.org Roll Skia from ad09ab099fe4 to 721bbd7c0340 (10 revisions) (flutter/engine#56063)
2024-10-23 aam@google.com Reland copy gen snapshot (flutter/engine#56061)
2024-10-23 skia-flutter-autoroll@skia.org Roll Dart SDK from 2a4b728f6a03 to 75c42f30af7a (1 revision) (flutter/engine#56060)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC codefu@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
Copy gen_snapshots using python's shutil.copy, avoid links
    
Default implementation of copy does it via hardlink, which seems to be
causing issues with Gatekeeper on mac.

This is reland of 63d8f5c with the fix
for the case when target is a link to the source, that was the cause for
revert c5ad234
    
BUG=flutter#154437
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.

2 participants