From eb9cbfe0b76e217743a18950693d90ebd3a20690 Mon Sep 17 00:00:00 2001
From: Christian Heller <c.heller@plomlompom.de>
Date: Sun, 20 Nov 2016 01:41:31 +0100
Subject: [PATCH] Document a bug and code problem to fix later.

---
 README.md               | 11 +++++++++--
 processor/feed.xml.do   |  2 ++
 processor/index.html.do |  5 +++--
 3 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/README.md b/README.md
index 80b13e8..eb2b318 100644
--- a/README.md
+++ b/README.md
@@ -20,8 +20,8 @@ Run ./test.sh.
 setup
 -----
 
-To set up a directory with symbolic links to the relevant .do files and other
-important files, run ./add_dir.sh DIRECTORY.
+To set up a directory with symbolic links to the relevant files in ./processor/,
+run ./add_dir.sh DIRECTORY.
 
 You can then enter the directory and run redo there. This will generate article
 .html files from all .md and .rst files, plus a ./index.html, and a ./feed.xml.
@@ -29,3 +29,10 @@ You can then enter the directory and run redo there. This will generate article
 also be generated a .uuid and a .intermediate file; furthermore, files with
 metadata used in ./feed.xml and ./index.html will be built and can be edited to
 customize the blog: ./url, ./author, ./uuid, ./title.) 
+
+bugs
+----
+
+Due to bad shell scripting in ./processor/index.html.do and
+./processor/feed.xml.do (see the FIXME notes there), source files whose names
+contain "$" break the redo processing.
diff --git a/processor/feed.xml.do b/processor/feed.xml.do
index 14ead59..4f67b27 100644
--- a/processor/feed.xml.do
+++ b/processor/feed.xml.do
@@ -35,6 +35,8 @@ printf "<id>urn:uuid:%s</id>\n" "$uuid"
 
 # Iterate through most recent entries (go by lastmod date of source files) to
 # build feed head "updated" element, and individual entries.
+# FIXME: This ls parsing is a bad way to loop through the sorted files. Besides,
+# $('\0') is a bashism.
 first_run=0
 files=`ls -1t *.rst *.md | head -10 | tr '\n' $'\0'`
 oldIFS="$IFS"
diff --git a/processor/index.html.do b/processor/index.html.do
index 2b71ffb..e436a4a 100644
--- a/processor/index.html.do
+++ b/processor/index.html.do
@@ -15,8 +15,9 @@ blog_title=`read_and_escape_file "$title_file" | head -1`
 printf "<title>%s</title>\n</head>\n<body>\n" "$blog_title"
 printf "<h1>%s</h1>\n<ul>\n" "$blog_title"
 
-# Iterate through entries sorted by lastmod of their source files, write entry
-# list. 
+# Iterate through entries sorted by lastmod of their source files, write entry.
+# FIXME: This ls parsing is a bad way to loop through the sorted files. Besides,
+# $('\0') is a bashism.
 first_run=0
 files=`ls -1t *.rst *.md | tr '\n' $'\0'`
 oldIFS="$IFS"
-- 
2.30.2