Skip to content

Commit 5f89d7f

Browse files
brianquinlanCommit Queue
authored andcommitted
Remove macOS as a platform that can run scripts in named pipes.
Bug:#28737 Change-Id: I977095f7e68b0a3e3514053295ce7d7154989e7e Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/329220 Reviewed-by: Alexander Aprelev <[email protected]> Commit-Queue: Brian Quinlan <[email protected]>
1 parent 1e2d1b2 commit 5f89d7f

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

tests/standalone/io/named_pipe_script_test.dart

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@
33
// BSD-style license that can be found in the LICENSE file.
44
// Testing file input stream, VM-only, standalone test.
55

6+
/// Tests the Dart scripts can be read from named pipes, e.g.
7+
/// `echo 'main(){print("hello, world");}' | dart /dev/fd/0`
8+
69
import "dart:convert";
710
import "dart:io";
811

@@ -11,9 +14,10 @@ import "package:expect/expect.dart";
1114

1215
main() async {
1316
asyncStart();
14-
// Reading a script from a named pipe is only supported on Linux and MacOS.
15-
if (!Platform.isLinux && !Platform.isMacOS) {
16-
print("This test is only supported on Linux and MacOS.");
17+
// Reading a script from a named pipe is only supported on Linux.
18+
// TODO(https://dartbug.com/26237): Enable support for additional platforms.
19+
if (!Platform.isLinux) {
20+
print("This test is only supported on Linux.");
1721
asyncEnd();
1822
return;
1923
}

0 commit comments

Comments
 (0)