Skip to content

[ci] Add gen_snapshot artifacts #227

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Dec 26, 2021
Merged

[ci] Add gen_snapshot artifacts #227

merged 5 commits into from
Dec 26, 2021

Conversation

@WonyoungChoi
Copy link

@WonyoungChoi WonyoungChoi commented Dec 23, 2021

Add jobs for building gen_snapshot on windows / macOS.

Added artifacts:

  • tizen-arm-release_linux-x64
  • tizen-arm-release_windows-x64
  • tizen-arm-release_darwin-x64
  • tizen-arm-profile_linux-x64
  • tizen-arm-profile_windows-x64
  • tizen-arm-profile_darwin-x64
  • tizen-arm64-release_linux-x64
  • tizen-arm64-release_windows-x64
  • tizen-arm64-release_darwin-x64
  • tizen-arm64-profile_linux-x64
  • tizen-arm64-profile_windows-x64
  • tizen-arm64-profile_darwin-x64

@WonyoungChoi WonyoungChoi force-pushed the gen_snapshot branch 2 times, most recently from aa5e71a to 43e0dde Compare December 23, 2021 09:12
@swift-kim
Copy link
Member

swift-kim commented Dec 23, 2021 

TIZEN_ENGINE_GITHUB_RUN_ID=1614508761 flutter-tizen precache --tizen --force

$ flutter-tizen run --release
Launching tizen/flutter/generated_main.dart on Tizen SM-R500 in release mode...
ProcessException: Found candidates, but lacked sufficient permissions to execute
"/Users/swift/Git/flutter-tizen/flutter/bin/cache/artifacts/engine/tizen-arm-release/darwin-x64/gen_snapshot".
  Command: /Users/swift/Git/flutter-tizen/flutter/bin/cache/artifacts/engine/tizen-arm-release/darwin-x64/gen_snapshot
Building a Tizen application in release mode...                    349ms
Exception: The build failed.

Worked on my watch (M1 Macbook as a host) after providing the executable permission to the gen_snapshot binary. Tested both release and profile modes.

@WonyoungChoi
Copy link
Author

WonyoungChoi commented Dec 23, 2021

😢 I will test on my Intel MacBook. but is it possible to run x64 executable on M1? What happens if you run the /Users/swift/Git/flutter-tizen/flutter/bin/cache/artifacts/engine/tizen-arm-release/darwin-x64/gen_snapshot executable directly?

@WonyoungChoi
Copy link
Author

WonyoungChoi commented Dec 23, 2021

Successful on Intel MacBook
Result:

xamarin@Kanghos-MacBook-Air myapp1 % flutter-tizen run --release
Launching tizen/flutter/generated_main.dart on Tizen rpi4 in release mode...
The TPK was signed with a default certificate. You can create one using Certificate Manager.
https://github.com/flutter-tizen/flutter-tizen/blob/master/doc/install-tizen-sdk.md#create-a-tizen-certificate
Building a Tizen application in release mode...                    42.4s
✓ Built build/tizen/tpk/com.example.myapp1-1.0.0.tpk (6.6MB).
Installing build/tizen/tpk/com.example.myapp1-1.0.0.tpk...          4.7s

Flutter run key commands.
h List all available interactive commands.
c Clear the screen
q Quit (terminate the application on the device).

swift-kim
swift-kim reviewed Dec 24, 2021
View reviewed changes
@swift-kim swift-kim linked an issue Dec 24, 2021 that may be closed by this pull request
Build and release our own gen_snapshots in the engine CI #224
Closed
swift-kim
swift-kim reviewed Dec 24, 2021
View reviewed changes
swift-kim
swift-kim approved these changes Dec 24, 2021
View reviewed changes
Copy link
Member

@swift-kim swift-kim left a comment

Choose a reason for hiding this comment

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

Thank you!

@swift-kim swift-kim merged commit c8a9118 into flutter-tizen:flutter-2.8.1-tizen Dec 26, 2021
@WonyoungChoi WonyoungChoi mentioned this pull request Dec 27, 2021
Merged
swift-kim pushed a commit that referenced this pull request Feb 7, 2022
@WonyoungChoi @swift-kim
[ci] Add gen_snapshot artifacts (#227)
8245e5a 
* [ci] Add gen_snapshot artifacts

* Remove printing environment values

* Apply shallow gclient sync

* Apply cache to macos-build job
swift-kim pushed a commit that referenced this pull request Feb 11, 2022
@WonyoungChoi @swift-kim
[ci] Add gen_snapshot artifacts (#227)
a1c4435 
* [ci] Add gen_snapshot artifacts

* Remove printing environment values

* Apply shallow gclient sync

* Apply cache to macos-build job
swift-kim pushed a commit that referenced this pull request May 12, 2022
@WonyoungChoi @swift-kim
[ci] Add gen_snapshot artifacts (#227)
328e81c 
* [ci] Add gen_snapshot artifacts

* Remove printing environment values

* Apply shallow gclient sync

* Apply cache to macos-build job
swift-kim pushed a commit that referenced this pull request Aug 5, 2022
@WonyoungChoi @swift-kim
[ci] Add gen_snapshot artifacts (#227)
5d25ee7 
* [ci] Add gen_snapshot artifacts

* Remove printing environment values

* Apply shallow gclient sync

* Apply cache to macos-build job
swift-kim pushed a commit that referenced this pull request Sep 1, 2022
@WonyoungChoi @swift-kim
[ci] Add gen_snapshot artifacts (#227)
8f1138d 
* [ci] Add gen_snapshot artifacts

* Remove printing environment values

* Apply shallow gclient sync

* Apply cache to macos-build job
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@swift-kim swift-kim swift-kim approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Build and release our own gen_snapshots in the engine CI

2 participants

@WonyoungChoi @swift-kim