Skip to content
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

Do not use pub.dart.snapshot #1394

Merged
merged 5 commits into from
Aug 28, 2021
Merged

Do not use pub.dart.snapshot #1394

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
48ff0de
Do not use pub.dart.snapshot
Aug 26, 2021
44fb1f5
Use instead of Pub is a package manager for Dart.
Aug 27, 2021
adf8fcb
Try fix daemon not starting on windows
Aug 27, 2021
68a3f49
Try fix test failures on windows
Aug 27, 2021
a368e5a
Remove outdated comment as well
Aug 27, 2021
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
8 changes: 4 additions & 4 deletions dwds/test/fixtures/utilities.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ Future<BuildDaemonClient> connectClient(String workingDirectory,
// the process will start in a disjoint cmd without access to
// STDIO. We also want to ensure the version of pub is consistent with
// the SDK that was used to launch webdev.
[dartPath, pubSnapshot]
..addAll(['run', 'build_runner', 'daemon'])
[dartPath, dartdevSnapshot]
annagrin marked this conversation as resolved.
Show resolved Hide resolved
..addAll(['pub', 'run', 'build_runner', 'daemon'])
..addAll(options),
logHandler: logHandler);

Expand All @@ -36,8 +36,8 @@ final String _sdkDir = (() {

final String dartSdkPath = _sdkDir;
final String dartPath = p.join(_sdkDir, 'bin', 'dart');
final String pubSnapshot =
p.join(_sdkDir, 'bin', 'snapshots', 'pub.dart.snapshot');
final String dartdevSnapshot =
p.join(_sdkDir, 'bin', 'snapshots', 'dartdev.dart.snapshot');
final String pubPath =
p.join(_sdkDir, 'bin', Platform.isWindows ? 'pub.bat' : 'pub');

Expand Down
2 changes: 2 additions & 0 deletions frontend_server_common/lib/src/utilities.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ final String dartSdkPath = _sdkDir;
final String dartPath = p.join(_sdkDir, 'bin', 'dart');
final String pubSnapshot =
p.join(_sdkDir, 'bin', 'snapshots', 'pub.dart.snapshot');
final String dartdevSnapshot =
p.join(_sdkDir, 'bin', 'snapshots', 'dartdev.dart.snapshot');
final String pubPath =
p.join(_sdkDir, 'bin', Platform.isWindows ? 'pub.bat' : 'pub');

Expand Down
4 changes: 2 additions & 2 deletions webdev/lib/src/daemon_client.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ Future<BuildDaemonClient> connectClient(String workingDirectory,
// the process will start in a disjoint cmd without access to
// STDIO. We also want to ensure the version of pub is consistent with
// the SDK that was used to launch webdev.
[dartPath, pubSnapshot]
..addAll(['run', 'build_runner', 'daemon'])
[dartPath, dartdevSnapshot]
..addAll(['pub', 'run', 'build_runner', 'daemon'])
..addAll(options),
logHandler: logHandler);

Expand Down
4 changes: 2 additions & 2 deletions webdev/lib/src/util.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ final String _sdkDir = (() {
})();

final String dartPath = p.join(_sdkDir, 'bin', 'dart');
final String pubSnapshot =
p.join(_sdkDir, 'bin', 'snapshots', 'pub.dart.snapshot');
final String dartdevSnapshot =
p.join(_sdkDir, 'bin', 'snapshots', 'dartdev.dart.snapshot');
final String pubPath =
p.join(_sdkDir, 'bin', Platform.isWindows ? 'pub.bat' : 'pub');