tests: ls/color-clear-to-eol: append NL to accommodate old sed
authorJim Meyering <meyering@redhat.com>
Mon, 7 Sep 2009 17:36:21 +0000 (19:36 +0200)
committerJim Meyering <meyering@redhat.com>
Mon, 7 Sep 2009 17:36:37 +0000 (19:36 +0200)
* tests/ls/color-clear-to-eol: Some vendor sed programs fail
to operate on lines that are not NL-terminated.
This affects at least Solaris 10's /bin/sed.
Reported by Pádraig Brady.

tests/ls/color-clear-to-eol

index 92cf8a8..fd65ced 100755 (executable)
@@ -30,11 +30,15 @@ e='\33'
 color_code='0;31;42'
 c_pre="$e[0m$e[${color_code}m"
 c_post="$e[0m$e[K\n$e[m"
-printf "$c_pre$long_name$c_post" > exp || framework_failure
+printf "$c_pre$long_name$c_post\n" > exp || framework_failure
 
 fail=0
 env TERM=xterm COLUMNS=80 LS_COLORS="*.foo=$color_code" TIME_STYLE=+T \
   ls -og --color=always $long_name > out || fail=1
+
+# Append a newline, to accommodate less-capable versions of sed.
+echo >> out || fail=1
+
 sed 's/.*T //' out > k && mv k out
 
 compare out exp || fail=1