bugfix html, bugfix reading final line of list

diff --git a/macros/revisions b/macros/revisions
index c6fb8fa2cb59b7d239b81f372515a5ed96ed3a55..2bd83f5ae487d2f4f38b17a2e7401f9cda2e61da 100755 (executable)
--- a/macros/revisions
+++ b/macros/revisions
@@ -13,14 +13,15 @@ elif [ ! "$(which git)" ]; then
   printf '<div class="macro revisions error">GIT is not available to handle revisioning.</div>'
 fi
 
-printf '<ul class="macro revisions">'
+printf '<ul class="macro revisions">\n'
   IFS="        "
-  git -C "$_DATA" log --date=format:"%a, %x %H:%M" \
-                      --pretty=format:"%h      %cd     %s%n" \
-                      -- "pages${page_abs}#page.md" \
-  | while read hash date message; do
+  { git -C "$_DATA" log --date=format:"%a, %x %H:%M" \
+                      --pretty=format:"%h      %cd     %s" \
+                      -- "pages${page_abs}#page.md"
+    printf '\n'
+  } | while read -r hash date message; do
     user="${message% @*}"; user="${user##*@ }"
-    printf '<a href="%s"><span class="hash">%s</span><span="date">%s</span><span class="user">%s</span></a>' \
+    printf '<li><span class="hash"><a href="%s">%s</a></span><span class="date">%s</span><span class="user">%s</span></li>\n' \
       "$(HTML "${page%/}/[revisions]/$hash")" "$(HTML "$hash")" "$(HTML "$date")" "$(HTML "$user")"
   done
-printf '</ul>'
+printf '</ul>\n'