Skip to content

Clean up tizen/BUILD.gn #150

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jul 23, 2021

Conversation

swift-kim
Copy link
Member

  • Do not depend on the build option --disable-desktop-embeddings. Use embedder_for_target to determine the target instead.
  • Merge embedder_for_profile and embedder_executable into a single template (embedder).
  • Be more explicit. e.g. The use_evas_gl_renderer argument is always required.

@@ -68,13 +68,13 @@ jobs:
flutter/tools/gn \
--target-os linux \
--linux-cpu $(arch) \
--no-goma \
Copy link
Member

Choose a reason for hiding this comment

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

👍

Copy link
Member

@bwikbs bwikbs left a comment

Choose a reason for hiding this comment

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

Good! 👍

@swift-kim swift-kim merged commit 15070da into flutter-tizen:flutter-2.2.1-tizen Jul 23, 2021
swift-kim added a commit that referenced this pull request Sep 27, 2021
* WIP 1

* WIP 2

* Clean up
swift-kim added a commit that referenced this pull request Nov 14, 2021
* WIP 1

* WIP 2

* Clean up
swift-kim added a commit that referenced this pull request Dec 9, 2021
* WIP 1

* WIP 2

* Clean up
swift-kim added a commit that referenced this pull request Dec 17, 2021
swift-kim added a commit that referenced this pull request Feb 7, 2022
swift-kim added a commit that referenced this pull request Feb 11, 2022
swift-kim added a commit that referenced this pull request May 12, 2022
swift-kim added a commit that referenced this pull request Aug 5, 2022
swift-kim added a commit that referenced this pull request Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants