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

Oops, allow files in opted-in third_party directories.#46897

Merged
matanlurey merged 3 commits into
flutter-team-archive:mainfrom
matanlurey:engine-thirdParty-gitIgnoreFix
Oct 13, 2023
Merged

Oops, allow files in opted-in third_party directories.#46897
matanlurey merged 3 commits into
flutter-team-archive:mainfrom
matanlurey:engine-thirdParty-gitIgnoreFix

Conversation

@matanlurey

Copy link
Copy Markdown
Contributor

@matanlurey matanlurey marked this pull request as ready for review October 13, 2023 18:33
@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 13, 2023
@cyanglaz

Copy link
Copy Markdown
Contributor

I tired this, didn't work, @jason-simmons found out that the below works. Unsure why

!txt/
!txt/**

@matanlurey

Copy link
Copy Markdown
Contributor Author

Ah, /* as an opt-out is better than *. Looks good now locally.

@cyanglaz cyanglaz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM
Verified locally it works

Comment thread third_party/.gitignore Outdated
!txt/
!web_locale_keymap/
!web_test_fonts/
!web_test_font/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like the removal of the s is a typo?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yup! Nice catch, thanks.

@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 13, 2023
@auto-submit

auto-submit Bot commented Oct 13, 2023

Copy link
Copy Markdown
Contributor

auto label is removed for flutter/engine/46897, due to - The status or check suite Linux linux_web_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.

@matanlurey matanlurey merged commit a8fb798 into flutter-team-archive:main Oct 13, 2023
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Oct 13, 2023
…6564)

Manual roll requested by jacksongardner@google.com

Cannot build log URL because revision "a4e62581525c" is invalid: Luci builds of "Mac mac_ios_engine" for a4e6258 was STARTED

2023-10-13 skia-flutter-autoroll@skia.org Manual roll Dart SDK from c8143a7c026f to 5844b34768ce (1 revision) (flutter-team-archive/engine#46909)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from c9601553b0f3 to 4bc4b4d22866 (1 revision) (flutter-team-archive/engine#46908)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from 8dc8e21a4dec to c9601553b0f3 (3 revisions) (flutter-team-archive/engine#46907)
2023-10-13 skia-flutter-autoroll@skia.org Roll Fuchsia Mac SDK from SDPa6SfaKvRRSUCij... to 4DwpR2CMJECZJ8EKz... (flutter-team-archive/engine#46903)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from b50d7f9aa743 to 8dc8e21a4dec (4 revisions) (flutter-team-archive/engine#46902)
2023-10-13 jonahwilliams@google.com [Impeller] fix clear color optimization for large subpasses. (flutter-team-archive/engine#46887)
2023-10-13 matanlurey@users.noreply.github.com Oops, allow files in opted-in third_party directories. (flutter-team-archive/engine#46897)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from e10e6765480a to b50d7f9aa743 (1 revision) (flutter-team-archive/engine#46899)
2023-10-13 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from pI1BVH08V0eG0M7sw... to wBb32R567S1alWbfn... (flutter-team-archive/engine#46898)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from pI1BVH08V0eG to wBb32R567S1a
  fuchsia/sdk/core/mac-amd64 from SDPa6SfaKvRR to 4DwpR2CMJECZ

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 jacksongardner@google.com,jsimmons@google.com,rmistry@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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

[engine] cannot add new files in flutter/third_party

3 participants