Skip to content

Comments

Windows build#94

Merged
bryanoltman merged 53 commits intoshorebird/devfrom
bo/windows-build
Jan 3, 2025
Merged

Windows build#94
bryanoltman merged 53 commits intoshorebird/devfrom
bo/windows-build

Conversation

@bryanoltman
Copy link

@bryanoltman bryanoltman commented Dec 17, 2024

Ready for review, with the caveat that there will be cleanup done as a followup PR to consolidate how/when ConfigureShorebird is called, with the eventual goal being a single ConfigureShorebird function.

Notes doc: https://docs.google.com/document/d/18t791EDSN03FbuZM9uE50CE36ZppF5i8PDaO7l6Bcms/edit?tab=t.0

Rust Setup

In engine/src/third_party/updater, run:

rustup target add x86_64-pc-windows-msvc
cargo build --release --target x86_64-pc-windows-msvc

Flutter Build

./flutter/tools/gn --runtime-mode release --no-rbe --no-lto
ninja -C .\out\host_release flutter/build/archives:windows_flutter

These commands were adapted from ci\builders\windows_host_engine.json

Other notes

The ninja build sometimes gets stuck in the following loop and I don't know why:

PS C:\Users\bryan\shorebirdtech\engine\src> ninja -C .\out\host_release flutter/build/archives:windows_flutter
ninja: Entering directory `.\out\host_release'
[0/1] Regenerating ninja files//out/host_release/environment.x64
HOMEDRIVE=C:
[0/1] Regenerating ninja files//out/host_release/environment.x64
HOMEDRIVE=C:
[0/1] Regenerating ninja files//out/host_release/environment.x64
HOMEDRIVE=C:
[0/1] Regenerating ninja files

@bryanoltman bryanoltman changed the title Bo/windows build Windows build Dec 17, 2024
@bryanoltman bryanoltman requested a review from felangel January 3, 2025 19:39
@eseidel
Copy link

eseidel commented Jan 3, 2025 via email

@bryanoltman bryanoltman merged commit 7e0f2a0 into shorebird/dev Jan 3, 2025
1 check passed
@bryanoltman bryanoltman deleted the bo/windows-build branch January 3, 2025 22:27
felangel added a commit that referenced this pull request Jan 14, 2025
* chore: squash all our changes to a single commit

See shorebird/dev-3.24.5 for prior commit history.

* fix: remove flutter/fml/size include

* fix: use shorebird buildroot

* fix: generate analyze snapshot for arm builds

* formatting

* fix: update how we generate analyze_snapshot (#92)

* fix: update how we generate analyze_snapshot

* formatting

* build windows

* Windows Support

* Format

* Make patching work

* Formatting

* Formatting

* Formatting

* Improve windows

* Formatting

* Formatting

* Formatting

* Revert unneeded changes

* Revert changes

* Bump updater rev

* Remove prints

* Cleanup

* fix deps

* Use correct local appdata path

* Formatting

* consolidate fetching build number and release version

* format

* remove unnecessary pointer

* Introduce ReleaseVersion struct

* formatting

* Formatting

* More formatting

* Comments

* Formatting

* More comments

* Formatting

* Remove print statements

* Remove comments

* Use in-out param for ConfigureShorebird

* formatting

* fix deps

* Remove extra include of updater.lib

* Cleanup

* cleanup

* Formatting

* Use ERROR_SUCCESS

* Do not use Error Success

* Add comment

* GetLocalAppDataPath feedback

* Refactor

---------

Co-authored-by: Felix Angelov <felix@shorebird.dev>
bryanoltman added a commit that referenced this pull request Feb 6, 2025
* chore: squash all our changes to a single commit

See shorebird/dev-3.24.5 for prior commit history.

* fix: remove flutter/fml/size include

* fix: use shorebird buildroot

* fix: generate analyze snapshot for arm builds

* formatting

* fix: update how we generate analyze_snapshot (#92)

* fix: update how we generate analyze_snapshot

* formatting

* build windows

* Windows Support

* Format

* Make patching work

* Formatting

* Formatting

* Formatting

* Improve windows

* Formatting

* Formatting

* Formatting

* Revert unneeded changes

* Revert changes

* Bump updater rev

* Remove prints

* Cleanup

* fix deps

* Use correct local appdata path

* Formatting

* consolidate fetching build number and release version

* format

* remove unnecessary pointer

* Introduce ReleaseVersion struct

* formatting

* Formatting

* More formatting

* Comments

* Formatting

* More comments

* Formatting

* Remove print statements

* Remove comments

* Use in-out param for ConfigureShorebird

* formatting

* fix deps

* Remove extra include of updater.lib

* Cleanup

* cleanup

* Formatting

* Use ERROR_SUCCESS

* Do not use Error Success

* Add comment

* GetLocalAppDataPath feedback

* Refactor

---------

Co-authored-by: Felix Angelov <felix@shorebird.dev>
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.

3 participants