Skip to content

Commit

Permalink
[Tizen] Fix Flutter 3.10 build
Browse files Browse the repository at this point in the history
  • Loading branch information
swift-kim committed May 18, 2023
1 parent 4abf603 commit 6531113
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 6 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,9 @@ jobs:

- name: Build
run: |
# Ignore unsupported linker option.
sed -i "/-Wl,--undefined-version/d" src/build/config/compiler/BUILD.gn
src/flutter/tools/gn \
--target-os linux \
--linux-cpu ${{ matrix.arch }} \
Expand Down Expand Up @@ -129,6 +132,7 @@ jobs:
- name: Run gclient sync
working-directory: C:\workspace\engine
shell: powershell
run: |
gclient config --name=src\flutter --unmanaged https://github.com/flutter-tizen/engine
gclient setdep --var=download_dart_sdk=False --deps-file=src/flutter/DEPS
Expand Down
4 changes: 1 addition & 3 deletions common/config.gni
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,4 @@ if (flutter_prebuilt_dart_sdk) {
# TODO: We can't build the engine artifacts for arm (32-bit) right now;
# see https://github.com/flutter/flutter/issues/74322
build_engine_artifacts =
flutter_build_engine_artifacts &&
(current_toolchain == host_toolchain ||
(is_linux && !is_chromeos && current_cpu != "arm") || is_mac || is_win)
flutter_build_engine_artifacts && current_toolchain == host_toolchain
3 changes: 2 additions & 1 deletion display_list/testing/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@ surface_provider_include_software = !is_android && !is_ios
# But, since benchmarks do not run on Windows and rendertests only
# runs on SW by default, this restriction currently only limits the
# ability to manually cross-check OpenGL on Windows for rendertests
surface_provider_include_gl = !is_fuchsia && !is_ios && !is_win && !is_mac
surface_provider_include_gl =
!is_fuchsia && !is_ios && !is_win && !is_mac && !is_linux

# TODO (https://github.com/flutter/flutter/issues/107357):
# impeller_enable_vulkan currently requires skia to not use VMA, which in turn
Expand Down
2 changes: 1 addition & 1 deletion impeller/geometry/half.h
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
#ifdef FML_OS_WIN
using InternalHalf = uint16_t;
#else
using InternalHalf = _Float16;
using InternalHalf = uint16_t;
#endif

namespace impeller {
Expand Down
2 changes: 1 addition & 1 deletion shell/config.gni
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,6 @@ declare_args() {
test_enable_metal = shell_enable_metal

# The Vulkan unittests are combined with the GL unittests.
test_enable_vulkan = is_fuchsia || shell_enable_gl
test_enable_vulkan = is_fuchsia
test_enable_software = shell_enable_software
}

0 comments on commit 6531113

Please sign in to comment.