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

Gn target to generate gen_snapshot for windows. #35986

Closed
godofredoc wants to merge 4 commits into from
Closed

Gn target to generate gen_snapshot for windows. #35986

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
209742e
Gn target to generate gen_snapshot for windows.
godofredoc Sep 8, 2022
3081fb3
Use target cpu.
godofredoc Sep 8, 2022
d604d34
Move to flutter/lib/snapshot file.
godofredoc Sep 8, 2022
4a50c14
Add zip gni dep.
godofredoc Sep 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Move to flutter/lib/snapshot file.
  • Loading branch information
@godofredoc
godofredoc committed Sep 8, 2022
commit d604d34bf314f5d2bd5f8e8c6c0638e6ca5259ee
13 changes: 0 additions & 13 deletions build/archives/BUILD.gn
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -274,16 +274,3 @@ if (is_mac) {
]
}
}

if (is_win) {
zip_bundle("archive_win_gen_snapshot") {
deps = [ "//third_party/dart/runtime/bin:gen_snapshot" ]
output = "$target_platform_name-$target_cpu-$flutter_runtime_mode/windows-x64.zip"
files = [
{
source = "$root_out_dir/gen_snapshot.exe"
destination = "gen_snapshot.exe"
},
]
}
}
13 changes: 13 additions & 0 deletions lib/snapshot/BUILD.gn
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -290,3 +290,16 @@ compile_platform("strong_platform") {
group("kernel_platform_files") {
public_deps = [ ":strong_platform" ]
}

if (is_win) {
zip_bundle("archive_win_gen_snapshot") {
Copy link
Member

@chinmaygarde chinmaygarde Sep 15, 2022

Choose a reason for hiding this comment

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

Who is the caller of this target definition? Is this patch complete?

Copy link
Contributor Author

@godofredoc godofredoc Sep 20, 2022

Choose a reason for hiding this comment

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

This was for experimenting with building the windows archives but we are currently blocked on arm support.

deps = [ "//third_party/dart/runtime/bin:gen_snapshot" ]
output = "$target_platform_name-$target_cpu-$flutter_runtime_mode/windows-x64.zip"
files = [
{
source = "$root_out_dir/gen_snapshot.exe"
destination = "gen_snapshot.exe"
},
]
}
}