s/while read/while read -r/

Author: uasi

boot2docker-depowify

@@ -5,7 +5,7 @@ set -u
 POW_HOST_ROOT=${POW_HOST_ROOT:-$HOME/Library/Application Support/Pow/Hosts}
 
 for_each() {
-    while read line; do
+    while read -r line; do
         [[ -n "$line" ]] || continue
         "$@" $line
     done
@@ -23,7 +23,7 @@ rm_proxy_conf() {
 
 stop_port_forwarding() {
     local port=$1
-    pgrep boot2docker | while read pid; do
+    pgrep boot2docker | while read -r pid; do
         local forwarder="boot2docker ssh -N -L $port:0.0.0.0:$port"
         if [[ $(ps -o command= $pid) = "$forwarder" ]]; then
             # Kill ssh spawned by boot2docker ssh.
@@ -68,7 +68,7 @@ filter_by_name() {
         cat
         return
     fi
-    while read line; do
+    while read -r line; do
         local name=$(cut -d' ' -f2 <<< $line)
         if grep -q " $name " <<< " $target_names "; then
             echo $line

boot2docker-powify

@@ -5,7 +5,7 @@ set -u
 POW_HOST_ROOT=${POW_HOST_ROOT:-$HOME/Library/Application Support/Pow/Hosts}
 
 for_each() {
-    while read line; do
+    while read -r line; do
         [[ -n "$line" ]] || continue
         "$@" $line
     done
@@ -23,7 +23,7 @@ start_port_forwarding() {
     local port=$1
     local forwarder="boot2docker ssh -N -L $port:0.0.0.0:$port"
     local forwarder_exists=
-    pgrep boot2docker | while read pid; do
+    pgrep boot2docker | while read -r pid; do
         if [[ $(ps -o command= $pid) = "$forwarder" ]]; then
             echo "Port $port is already forwarded"
             return 99
@@ -53,7 +53,7 @@ filter_by_name() {
         cat
         return
     fi
-    while read line; do
+    while read -r line; do
         local name=$(cut -d' ' -f2 <<< $line)
         if grep -q " $name " <<< " $target_names "; then
             echo $line