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

[Windows] Use 'ninja' instead of 'ninja.exe' #39326

Merged
merged 1 commit into from
Feb 2, 2023

Conversation

loic-sharma
Copy link
Member

@loic-sharma loic-sharma commented Feb 1, 2023

The engine's Visual Studio integration is currently broken. See: https://groups.google.com/a/chromium.org/g/chromium-dev/c/027jM6DLkIk

For Developers who build on Windows, instead of calling “ninja.exe”, please call “ninja” to resolve the depot_tools/ninja.bat wrapper.

Note that the Visual Studio integration won't be fixed until depot_tools#4215121 lands.

Part of: flutter/flutter#119760

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@loic-sharma loic-sharma marked this pull request as ready for review February 1, 2023 23:36
@loic-sharma
Copy link
Member Author

@cbracken @godofredoc Please let me know if there are other folks I should add as reviewers.

@zanderso zanderso merged commit 388890a into flutter:main Feb 2, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Feb 2, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Feb 2, 2023
…119830)

* e64f3264c Add missing file to artifacts.zip and win platform. (flutter/engine#39310)

* a19cf360a Roll Skia from 934ef0660509 to c3c5c6f61421 (1 revision) (flutter/engine#39332)

* 76dc4f989 Roll Skia from c3c5c6f61421 to 532c04c00ad7 (1 revision) (flutter/engine#39333)

* 8a74e633e Roll Dart SDK from 99014717b633 to 11d97405a447 (3 revisions) (flutter/engine#39337)

* 67662080e Roll Fuchsia Mac SDK from j5rQsxkO2VJrX55Yj... to rqjJA8z_K0gdGcsvA... (flutter/engine#39338)

* 388890a98 [Windows] Use 'ninja' instead of 'ninja.exe' (flutter/engine#39326)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants