From 20ff180ae83460f2f8f83be8aaddbb875e8daa59 Mon Sep 17 00:00:00 2001 From: Chris Bracken Date: Wed, 12 Jan 2022 18:21:05 -0800 Subject: [PATCH] Update Flutter desktop doc URL in error message (#96559) When desktop support is not present in an existing project, certain flutter tool commands raise an error that direct the user to documentation on how to add desktop support to an existing Flutter project. In a recent revamp of the webside, the URL was very slightly changed (flutter.dev -> docs.flutter.dev). This updates the error message to output the new URL. Issue: https://github.com/flutter/flutter/issues/94398 --- packages/flutter_tools/lib/src/linux/build_linux.dart | 2 +- packages/flutter_tools/lib/src/macos/build_macos.dart | 2 +- packages/flutter_tools/lib/src/windows/build_windows.dart | 4 ++-- .../test/commands.shard/hermetic/build_linux_test.dart | 2 +- .../test/commands.shard/hermetic/build_macos_test.dart | 2 +- .../test/commands.shard/hermetic/build_windows_test.dart | 4 ++-- 6 files changed, 8 insertions(+), 8 deletions(-) diff --git a/packages/flutter_tools/lib/src/linux/build_linux.dart b/packages/flutter_tools/lib/src/linux/build_linux.dart index 49f14a1805..81c86d5122 100644 --- a/packages/flutter_tools/lib/src/linux/build_linux.dart +++ b/packages/flutter_tools/lib/src/linux/build_linux.dart @@ -38,7 +38,7 @@ Future buildLinux( target ??= 'lib/main.dart'; if (!linuxProject.cmakeFile.existsSync()) { throwToolExit('No Linux desktop project configured. See ' - 'https://flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' + 'https://docs.flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' 'to learn about adding Linux support to a project.'); } diff --git a/packages/flutter_tools/lib/src/macos/build_macos.dart b/packages/flutter_tools/lib/src/macos/build_macos.dart index 335f63415f..c2177505ad 100644 --- a/packages/flutter_tools/lib/src/macos/build_macos.dart +++ b/packages/flutter_tools/lib/src/macos/build_macos.dart @@ -31,7 +31,7 @@ Future buildMacOS({ }) async { if (!flutterProject.macos.xcodeWorkspace.existsSync()) { throwToolExit('No macOS desktop project configured. ' - 'See https://flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' + 'See https://docs.flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' 'to learn about adding macOS support to a project.'); } diff --git a/packages/flutter_tools/lib/src/windows/build_windows.dart b/packages/flutter_tools/lib/src/windows/build_windows.dart index 1d7a269896..6422d39d72 100644 --- a/packages/flutter_tools/lib/src/windows/build_windows.dart +++ b/packages/flutter_tools/lib/src/windows/build_windows.dart @@ -32,7 +32,7 @@ Future buildWindows(WindowsProject windowsProject, BuildInfo buildInfo, { if (!windowsProject.cmakeFile.existsSync()) { throwToolExit( 'No Windows desktop project configured. See ' - 'https://flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' + 'https://docs.flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' 'to learn about adding Windows support to a project.'); } @@ -124,7 +124,7 @@ Future buildWindowsUwp(WindowsUwpProject windowsProject, BuildInfo buildIn if (!windowsProject.existsSync()) { throwToolExit( 'No Windows UWP desktop project configured. See ' - 'https://flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' + 'https://docs.flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' 'to learn about adding Windows support to a project.', ); } diff --git a/packages/flutter_tools/test/commands.shard/hermetic/build_linux_test.dart b/packages/flutter_tools/test/commands.shard/hermetic/build_linux_test.dart index 9714f74059..0d73157cd1 100644 --- a/packages/flutter_tools/test/commands.shard/hermetic/build_linux_test.dart +++ b/packages/flutter_tools/test/commands.shard/hermetic/build_linux_test.dart @@ -114,7 +114,7 @@ void main() { expect(createTestCommandRunner(command).run( const ['build', 'linux', '--no-pub'] ), throwsToolExit(message: 'No Linux desktop project configured. See ' - 'https://flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' + 'https://docs.flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' 'to learn about adding Linux support to a project.')); }, overrides: { Platform: () => linuxPlatform, diff --git a/packages/flutter_tools/test/commands.shard/hermetic/build_macos_test.dart b/packages/flutter_tools/test/commands.shard/hermetic/build_macos_test.dart index febf95159d..4e5cc455d3 100644 --- a/packages/flutter_tools/test/commands.shard/hermetic/build_macos_test.dart +++ b/packages/flutter_tools/test/commands.shard/hermetic/build_macos_test.dart @@ -131,7 +131,7 @@ void main() { expect(createTestCommandRunner(command).run( const ['build', 'macos', '--no-pub'] ), throwsToolExit(message: 'No macOS desktop project configured. See ' - 'https://flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' + 'https://docs.flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' 'to learn about adding macOS support to a project.')); }, overrides: { Platform: () => macosPlatform, diff --git a/packages/flutter_tools/test/commands.shard/hermetic/build_windows_test.dart b/packages/flutter_tools/test/commands.shard/hermetic/build_windows_test.dart index 1244d02c3c..a85bec4658 100644 --- a/packages/flutter_tools/test/commands.shard/hermetic/build_windows_test.dart +++ b/packages/flutter_tools/test/commands.shard/hermetic/build_windows_test.dart @@ -159,7 +159,7 @@ void main() { expect(createTestCommandRunner(command).run( const ['windows', '--no-pub'] ), throwsToolExit(message: 'No Windows desktop project configured. See ' - 'https://flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' + 'https://docs.flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' 'to learn about adding Windows support to a project.')); }, overrides: { Platform: () => windowsPlatform, @@ -489,7 +489,7 @@ C:\foo\windows\runner\main.cpp(17,1): error C2065: 'Baz': undeclared identifier expect(createTestCommandRunner(command).run( const ['winuwp', '--no-pub'] ), throwsToolExit(message: 'No Windows UWP desktop project configured. See ' - 'https://flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' + 'https://docs.flutter.dev/desktop#add-desktop-support-to-an-existing-flutter-app ' 'to learn about adding Windows support to a project.')); }, overrides: { Platform: () => windowsPlatform,