Merge pull request #22 from mbarbin/interactive-pipe-stdout

Fix Interactive Test Commands (`pipe-stdout`)
mbarbin · Dec 28, 2024 · a267132 · a267132
2 parents 248d765 + 5b27842
commit a267132
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 18 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,3 +1,17 @@
+## 0.3.3 (unreleased)
+
+### Added
+
+### Changed
+
+### Deprecated
+
+### Fixed
+
+- Replace `pipe-stdout` constructs by `bash` when they are expected to be interactive (#22, @mbarbin).
+
+### Removed
+
 ## 0.3.2 (2024-09-29)
 
 ### Changed

diff --git a/project/visa/circuit/dune b/project/visa/circuit/dune
@@ -112,10 +112,8 @@
   initial-memory.txt
   (package bopkit))
  (action
-  (pipe-stdout
-   (run %{bin:bopkit} simu %{dep:visa.bop} --output-only-on-change)
-   (run %{bin:bopkit} digital-calendar map-raw-input)
-   (run %{bin:bopkit} digital-calendar display --no-output))))
+  (bash
+   "%{bin:bopkit} simu %{dep:visa.bop} --output-only-on-change | %{bin:bopkit} digital-calendar map-raw-input | %{bin:bopkit} digital-calendar display --no-output")))
 
 (rule
  (alias visa-simulation)
@@ -124,10 +122,8 @@
   initial-memory.txt
   (package bopkit))
  (action
-  (pipe-stdout
-   (run %{bin:bopkit} simu %{dep:visa.bop} --output-only-on-change)
-   (run %{bin:bopkit} simu %{dep:calendar-output.bop} -p)
-   (run %{bin:bopkit} digital-calendar display --no-output))))
+  (bash
+   "%{bin:bopkit} simu %{dep:visa.bop} --output-only-on-change | %{bin:bopkit} simu %{dep:calendar-output.bop} -p | %{bin:bopkit} digital-calendar display --no-output")))
 
 (rule
  (target div10.txt)

diff --git a/stdlib/7-segment/bin/dune b/stdlib/7-segment/bin/dune
@@ -28,21 +28,17 @@
 (rule
  (alias digital-watch-display-test)
  (action
-  (pipe-stdout
-   (run %{dep:digital_watch.exe} gen-input)
-   (run %{dep:digital_watch_display.exe} --no))))
+  (bash
+   "./%{dep:digital_watch.exe} gen-input | ./%{dep:digital_watch_display.exe} --no")))
 
 (rule
  (alias digital-calendar-display-test)
  (action
-  (pipe-stdout
-   (run %{dep:digital_calendar.exe} gen-input)
-   (run %{dep:digital_calendar_display.exe} --no))))
+  (bash
+   "./%{dep:digital_calendar.exe} gen-input | ./%{dep:digital_calendar_display.exe} --no")))
 
 (rule
  (alias digital-calendar-display-raw-input-test)
  (action
-  (pipe-stdout
-   (run %{dep:digital_calendar.exe} gen-raw-input)
-   (run %{dep:digital_calendar.exe} map-raw-input)
-   (run %{dep:digital_calendar_display.exe} --no))))
+  (bash
+   "./%{dep:digital_calendar.exe} gen-raw-input | ./%{dep:digital_calendar.exe} map-raw-input | ./%{dep:digital_calendar_display.exe} --no")))