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

Conversation

@jason-simmons
Copy link
Member

@jason-simmons jason-simmons commented Jul 28, 2023

The Vulkan validation layer headers will not build if the Android API level is 23 or lower.

Currently the engine buildroot's default Android API level is 23. The API level must be overridden for any configuration where Vulkan validation is enabled.

…automatically enabled for android_debug_unopt_arm64 builds
@jason-simmons jason-simmons requested a review from matanlurey July 28, 2023 20:07
@matanlurey
Copy link
Contributor

Thanks! Do you mind editing the PR description and add 1-2 sentences on what problem this solves for posterity?

@jason-simmons
Copy link
Member Author

Thanks! Do you mind editing the PR description and add 1-2 sentences on what problem this solves for posterity?

done

@matanlurey
Copy link
Contributor

That makes sense, thanks for fixing this!

@jason-simmons jason-simmons added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 28, 2023
@auto-submit auto-submit bot merged commit b4bf592 into flutter:main Jul 28, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 28, 2023
…ayers are automatically enabled for android_debug_unopt_arm64 builds (flutter/engine#44116)
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jul 28, 2023
…131519)

flutter/engine@da3721a...b4bf592

2023-07-28 [email protected] Set the required Android API level when Vulkan validation layers are automatically enabled for android_debug_unopt_arm64 builds (flutter/engine#44116)
2023-07-28 [email protected] Roll Dart SDK from e9bdbe308478 to 69c96c505f05 (1 revision) (flutter/engine#44118)

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 [email protected],[email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
LouiseHsu pushed a commit to LouiseHsu/flutter that referenced this pull request Jul 31, 2023
…lutter#131519)

flutter/engine@da3721a...b4bf592

2023-07-28 [email protected] Set the required Android API level when Vulkan validation layers are automatically enabled for android_debug_unopt_arm64 builds (flutter/engine#44116)
2023-07-28 [email protected] Roll Dart SDK from e9bdbe308478 to 69c96c505f05 (1 revision) (flutter/engine#44118)

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 [email protected],[email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
vashworth pushed a commit to vashworth/flutter that referenced this pull request Aug 2, 2023
…lutter#131519)

flutter/engine@da3721a...b4bf592

2023-07-28 [email protected] Set the required Android API level when Vulkan validation layers are automatically enabled for android_debug_unopt_arm64 builds (flutter/engine#44116)
2023-07-28 [email protected] Roll Dart SDK from e9bdbe308478 to 69c96c505f05 (1 revision) (flutter/engine#44118)

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 [email protected],[email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
gaaclarke pushed a commit to gaaclarke/engine that referenced this pull request Aug 30, 2023
…automatically enabled for android_debug_unopt_arm64 builds (flutter#44116)

The Vulkan validation layer headers will not build if the Android API level is 23 or lower.

Currently the engine buildroot's default Android API level is 23.  The API level must be overridden for any configuration where Vulkan validation is enabled.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants