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

[fuchsia] Add libbackend_fuchsia_globals.so to common_libs #49484

Merged
merged 2 commits into from
Jan 4, 2024

Conversation

jrwang
Copy link
Contributor

@jrwang jrwang commented Jan 3, 2024

The libbackend_fuchsia_globals.so is now required by libsyslog.so, so include it in the list of Fuchsia common_libs.

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.

b/315973146

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides].
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or the PR is [test-exempt]. See [testing the engine] for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the [CLA].
  • All existing and new tests are passing.

@mbrase mbrase force-pushed the backend_fuchsia_globals branch from d187e83 to d742b97 Compare January 3, 2024 17:23
@mbrase
Copy link
Contributor

mbrase commented Jan 3, 2024

I created a flutter issue at flutter/flutter#140891

@mbrase mbrase force-pushed the backend_fuchsia_globals branch from 860e5d0 to eec970a Compare January 3, 2024 17:46
skia-flutter-autoroll and others added 2 commits January 3, 2024 22:59
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/fuchsia-linux-sdk-flutter-engine
Please CC chinmaygarde@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
The libbackend_fuchsia_globals.so is now required by libsyslog.so, so
include it in the list of Fuchsia common_libs.
@mbrase mbrase force-pushed the backend_fuchsia_globals branch from eec970a to 243fd8f Compare January 3, 2024 23:01
Copy link

@cligh cligh left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@mbrase
Copy link
Contributor

mbrase commented Jan 3, 2024

This is ready for merging. I am rolling the 16.20231205.1.1 SDK along with my fix due to a hard transition. I ran into another breakage when I attempted to roll a more recent SDK, so I'll fix that in a followup PR.

@mbrase mbrase force-pushed the backend_fuchsia_globals branch 3 times, most recently from efcdd1f to 243fd8f Compare January 3, 2024 23:45
@cligh cligh merged commit 28ae9e3 into flutter:main Jan 4, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jan 4, 2024
…140936)

flutter/engine@ab4098c...28ae9e3

2024-01-04 jonnywang@google.com [fuchsia] Add libbackend_fuchsia_globals.so to common_libs (flutter/engine#49484)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from CGyrUgA1M1yF to Wgogr8K1YZdo

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 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
@mbrase mbrase deleted the backend_fuchsia_globals branch January 23, 2024 19:18
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.

4 participants