fix the hung up bug of --dry-run which was introduced in v0.5.1.

shenwei356 · shenwei356 · commit 4833e4cf2027 · 2023-07-16T10:56:10.000+08:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,5 @@
+- rush v0.5.2
+    - fix the hung up bug of `--dry-run` which was introduced in v0.5.1.
 - rush v0.5.1
     - graceful handling of Ctrl+C.
     - `-c`: fix a bug -- some commands are recorded even after unsuccessfully running or interrupting with Ctrl+C. [#47](https://github.com/shenwei356/rush/issues/47)
diff --git a/helper.go b/helper.go
@@ -30,7 +30,7 @@ import (
 )
 
 // VERSION of this package
-const VERSION = "0.5.1"
+const VERSION = "0.5.2"
 
 func isStdin(file string) bool {
 	return file == "-"
diff --git a/process/process.go b/process/process.go
@@ -123,6 +123,7 @@ func (c *Command) Run(opts *Options, tryNumber int) (chan string, error) {
 		ch <- c.Cmd + "\n"
 		close(ch)
 		c.finishSendOutput = true
+		close(c.Executed)
 		return ch, nil
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+- rush v0.5.2`
	`2`	+ - fix the hung up bug of `--dry-run` which was introduced in v0.5.1.
`1`	`3`	`- rush v0.5.1`
`2`	`4`	`- graceful handling of Ctrl+C.`
`3`	`5`	- `-c`: fix a bug -- some commands are recorded even after unsuccessfully running or interrupting with Ctrl+C. [#47](https://github.com/shenwei356/rush/issues/47)
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ import (`
`30`	`30`	`)`
`31`	`31`
`32`	`32`	`// VERSION of this package`
`33`		`-const VERSION = "0.5.1"`
	`33`	`+const VERSION = "0.5.2"`
`34`	`34`
`35`	`35`	`func isStdin(file string) bool {`
`36`	`36`	`return file == "-"`
Original file line number	Diff line number	Diff line change
`@@ -123,6 +123,7 @@ func (c Command) Run(opts Options, tryNumber int) (chan string, error) {`
`123`	`123`	`ch <- c.Cmd + "\n"`
`124`	`124`	`close(ch)`
`125`	`125`	`c.finishSendOutput = true`
	`126`	`+ close(c.Executed)`
`126`	`127`	`return ch, nil`
`127`	`128`	`}`
`128`	`129`