Makes lots of changes. (See full log.)

- Converts Makefile to UNIX line termination.

Makefiles cannot be formatted with DOS CRLF because, otherwise, we
could not escape \n to continue on a second line.  We end up escaping
\r and not \n.

- Fixes remove-account!

Procedure delete-if ``may modify sequence'', but we cannot be sure it
will modify it.  There are cases in which it does and there are cases
in which it doesn't.  Seeing it did modify in one case, I incorrectly
assumed it would modify in all cases---such is life.  Since I do want
to modify it always, I wrote delete-if*, which in calls setf to be
sure the list is overwritten.

- Avoids (load'ing "~/.sbclrc") and, instead, does what Quicklisp does.

It's not always the case that ~/.sbclrc exists.  But Quicklisp in each
installation knows what to do.  So we do what Quicklisp does in each
script that we use.

- Frees us from using sb-ext:{file-exists,file-does-not-exist}

Turns out SBCL 1.2.4.debian doesn't have these symbols.

- Adds command-line parsing.
- Adds install target to Makefile.
- Rewrites Makefile with a new strategy.
- Adds the daemon-tls/ service example.
- Adds the daemon service example.
- Convers all text files to LF instead of CRLF.

Makefile

@@ -1,79 +1,8 @@
-SHELL=/bin/sh
+default: loop
-REMOTE=dbastos@antartida.xyz
+
-REMOTE_LIB_PATH=quicklisp/local-projects
+loop: loop.asd loop.lisp scripts/build-exe.lisp
-REMOTE_EXE_PATH=loop-test
+	sbcl --script scripts/build-exe.lisp
-SERVICE_NAME=loop-test
+
-
+install: loop
-default:
+	mkdir -p `head -1 conf-home` && \
-	@echo "Sorry. You need to read the Makefile to know what I can make for you."
+		cp -R loop accounts.lisp groups scripts `head -1 conf-home`
-
-all: loop.lisp build-exe.lisp exe run \
-migrate-add-creation-date.lisp cron-remove-inactive-users.lisp
-
-live: all remote-copy # remote-build-exe
-
-remote-copy:
-	scp loop.asd loop.lisp \
-	   $(REMOTE):$(REMOTE_LIB_PATH)/loop
-	scp build-exe.lisp \
-	   $(REMOTE):$(REMOTE_EXE_PATH)/
-	scp migrate-add-creation-date.lisp \
-	   $(REMOTE):$(REMOTE_EXE_PATH)/
-	scp cron-remove-inactive-users.lisp \
-	   $(REMOTE):$(REMOTE_EXE_PATH)/
-
-sync-users:
-	scp $(REMOTE):$(REMOTE_EXE_PATH)/accounts.lisp .
-
-remote-build-exe:
-	plink -ssh $(REMOTE) cd $(REMOTE_EXE_PATH)/ && \
-	   sbcl --script build-exe.lisp && \
-           echo "Executable built."
-
-remote-migrate-account-creation:
-	plink -ssh $(REMOTE) cd $(REMOTE_EXE_PATH)/ && \
-	   sbcl --script migrate-add-creation-date.lisp
-
-remote-cron-remove-inactive-users:
-	plink -ssh $(REMOTE) cd $(REMOTE_EXE_PATH)/ && \
-	   sbcl --script remote-cron-remove-inactive-users.lisp
-
-livedoc:
-	echo loop.nw | python peat -C 'make loop.pdf'
-
-run: loop.nw
-	(any tangle -Rrun < loop.nw > run.tmp || (rm run.tmp && exit 1)) && \
-	mv run.tmp run.lisp && \
-	chmod 0755 run	
-
-loop.tex: loop.nw
-	any weave -delay -index loop.nw > loop.tex
-
-loop.pdf: loop.tex
-	latexmk -pdf loop
-
-loop.lisp: loop.nw
-	(any tangle -Rloop.lisp < loop.nw > loop.tmp || (rm loop.tmp && exit 1)) && \
-	mv loop.tmp loop.lisp
-
-build-exe.lisp: loop.nw
-	(any tangle -Rbuild-exe.lisp < loop.nw > build-exe.tmp || (rm build-exe.tmp && exit 1)) && \
-	    mv build-exe.tmp build-exe.lisp
-
-build-index-from-fs.lisp: loop.nw
-	(any tangle -Rbuild-index-from-fs.lisp < loop.nw > build-index-from-fs.tmp || (rm build-index-from-fs.tmp && exit 1)) && \
-	    mv build-index-from-fs.tmp build-index-from-fs.lisp
-
-cron-remove-inactive-users.lisp: loop.nw
-	(any tangle -Rcron-remove-inactive-users.lisp < loop.nw > cron-remove-inactive-users.tmp || (rm cron-remove-inactive-users.tmp && exit 1)) && \
-	    mv cron-remove-inactive-users.tmp cron-remove-inactive-users.lisp
-
-migrate-add-creation-date.lisp: loop.nw
-	(any tangle -Rmigrate-add-creation-date.lisp < loop.nw > migrate-add-creation-date.tmp || (rm migrate-add-creation-date.tmp && exit 1)) && \
-	    mv migrate-add-creation-date.tmp migrate-add-creation-date.lisp
-
-exe: loop.lisp build-exe.lisp
-	sbcl --script build-exe.lisp && echo "Executable built okay."
-
-service: run
-	sudo ln -s $(pwd) /service/$(SERVICE_NAME)

README

@@ -0,0 +1,160 @@
+(*) Introduction
+
+LOOP is an NNTP server written in Common Lisp.
+
+(*) Assumptions
+
+We assume
+
+  - you run SBCL
+
+  - you have Quicklisp installed and knows how to use it
+
+  - you know how to use a TCP server such as 
+
+      https://cr.yp.to/ucspi-tcp.html
+
+  - you know how to manage a daemon with 
+
+      https://cr.yp.to/daemontools.html
+
+  - you have git and knows how to use it
+
+(*) How to install it
+
+LOOP is not in the Quicklisp repository, so we'll instruct you to
+install it as a local project.  Go to
+
+  ~/quicklisp/local-projects/
+
+and say 
+
+  $ git clone https://git.antartida.xyz/loop/srv loop
+  $ cd loop
+  $ echo /path/to/loop/home > conf-home
+  $ make install
+
+If you just installed SBCL and quicklisp, the build might take a
+little while due to downloading dependencies.  Be patient.
+
+(*) Systems with no installation issues
+
+We installed LOOP just fine on
+
+  FreeBSD 14.1, 14.2 with SBCL 2.4.9.
+  Debian GNU/Linux 8.11 codename jessie with SBCL 1.2.4.debian.
+
+(*) Systems with installation issues
+
+We installed LOOP on Ubuntu 24.04 (24.01.1 LTS) codename noble with
+SBCL 2.2.9.debian.  We found that CLSQL could not load the shared
+object libsqlite3.so because ``apt install libsqlite3'' installs the
+library at 
+
+  /usr/lib/x86_64-linux-gnu/libsqlite3.so.0.8.6
+
+with a symbolic link to libsqlite3.so.0, but not to libsqlite3.so.
+SBCL is trying to load libsqlite3.so, so a solution is to just tell
+your system to
+
+  ln -s libsqlite3.so.0 libsqlite3.so
+
+at /usr/lib/x86_64-linux-gnu.
+
+(*) Running LOOP
+
+First, try it out.
+
+--8<-------------------------------------------------------->8---
+$ cd /path/to/loop/home
+$ ./loop --help
+NAME:
+  loop - An NNTP server for a circle of friends.
+
+USAGE:
+  loop [options] [arguments ...]
+
+OPTIONS:
+      --change-passwd <VALUE>   <username> <new-password> changes password
+      --create-account <VALUE>  <username> <invited-by> creates a new account
+      --help                    display usage information and exit
+      --logging                 turn on debug logging on stderr
+      --version                 display version and exit
+  -l, --list-accounts           lists accounts
+  -r, --repl                    run a REPL on port 4006
+  -s, --server                  runs NNTP server reading from stdout
+
+AUTHORS:
+  Circling Skies <loop@antartida.xyz>
+
+LICENSE:
+  GPL v3
+--8<-------------------------------------------------------->8---
+
+You can talk to the NNTP server with -s:
+
+--8<-------------------------------------------------------->8---
+$ ./loop -s
+200 Welcome! Say ``help'' for a menu.
+quit
+205 Good-bye.
+--8<-------------------------------------------------------->8---
+
+It's time to create an account for you.  Whenever you run loop, make
+sure you're in its home directory because it will look for the file
+accounts.lisp always relatively to the current working directory of
+the process.  The same applies if you set up a cron job later
+on---make sure the job, too, sets LOOP's home directory as its current
+working directory.
+
+(*) Create your account
+
+LOOP requires authentication for most things, so you should create an
+account for you right away.  Accounts are kept in accounts.lisp in
+your installation directory.  Every time you create an account, you
+must specify who is inviting this new account into the loop---because
+we keep a tree of accounts.  The root account is called anonymous, so
+your first account must be invited by the anonymous account.  So you
+can say
+
+  ./loop --create-account you anonymous
+
+The anonymous account has no special power; it exists solely because
+the graph of accounts needs a root.
+
+(*) How to expose LOOP to the network
+
+Run your TCP server of choice.  For instance, if you're using djb's
+tcpserver and would like LOOP to listen on port 1024, tell your shell
+
+--8<-------------------------------------------------------->8---
+$ tcpserver -v -HR 0.0.0.0 1024 ./loop -s
+tcpserver: status: 0/40
+--8<-------------------------------------------------------->8---
+
+Using another terminal, telnet to your host on port 1024:
+
+--8<-------------------------------------------------------->8---
+$ telnet localhost 1024
+Trying 127.0.0.1...
+Connected to antartida.xyz.
+Escape character is '^]'.
+200 Welcome! Say ``help'' for a menu.
+quit
+205 Good-bye.
+Connection closed by foreign host.
+--8<-------------------------------------------------------->8---
+
+Directories daemon/ and daemon-tls/ in LOOP's source code have sample
+scripts to use with djb's tcpserver and daemontools.  If you have
+never done this, it will be better education if you learn to use
+daemontools and ucspi-tcp before going live with a LOOP community.
+It's easy and fun.
+
+(*) Cron jobs
+
+If you'd like to remove inactive accounts, we wrote
+scripts/cron-remove-inactive-users.lisp.  Her's our crontab:
+
+$ crontab -l
+@daily cd /path/to/loop/home && sbcl --script scripts/cron-remove-inactive-users.lisp

build-exe.lisp

@@ -1,5 +0,0 @@
-(load "~/.sbclrc")
-(ql:quickload :loop :silent t)
-(sb-ext:save-lisp-and-die #P"loop.exe"
-                         :toplevel #'loop:main
-                         :executable t)

build-index-from-fs.lisp

@@ -1,7 +0,0 @@
-(load "~/.sbclrc")
-(ql:quickload :loop :silent t)
-(in-package #:loop)
-(connect-index! "message-id.db")
-(remake-index!)
-(index-from-fs!)
-(format t "Index built.~%")

cron-remove-inactive-users.lisp

@@ -1,8 +0,0 @@
-(load "~/.sbclrc")
-(ql:quickload :loop :silent t)
-(in-package #:loop)
-;; (format t *default-pathname-defaults*)
-(read-accounts!)
-(connect-index! "message-id.db")
-(remove-inactive-users!)
-(write-accounts!)

daemon-tls/README

@@ -0,0 +1,4 @@
+In this service example, we're using tlswrapper by Jan Mojžíš, the
+source of which you can find at 
+
+  https://github.com/janmojzis/tlswrapper

daemon-tls/log/run

@@ -0,0 +1,3 @@
+#!/bin/sh
+echo loop-tls-logger
+exec /usr/bin/logger -i -t loop-tls

daemon-tls/run

@@ -0,0 +1,7 @@
+#!/bin/sh
+echo loop-tls
+cd /path/to/loop
+exec tcpserver -HR 0.0.0.0 563 \
+     /usr/bin/tlswrapper -f \
+     /usr/local/etc/letsencrypt/live/mydomain/cert-priv1.pem \
+     /path/to/loop

daemon/log/run

@@ -0,0 +1,3 @@
+#!/bin/sh
+echo loop-log
+exec /usr/bin/logger -i -t loop

daemon/run

@@ -0,0 +1,4 @@
+#!/bin/sh
+echo loop
+cd /path/to/your/loop
+exec /path/to/tcpserver -HR 0.0.0.0 119 /path/to/your/loop

groups/local.control.news/1

@@ -1,7 +1,7 @@
-Date: 2024-03-07 21:44:31 GMT-3
+Date: 2024-03-07 21:44:31 GMT-3
-Message-Id: <edjocyeqzqqhnswlbrbo@loop>
+Message-Id: <edjocyeqzqqhnswlbrbo@loop>
-From: Loop
+From: Loop
-Subject: let there be light
+Subject: let there be light
-Newsgroups: local.control.news
+Newsgroups: local.control.news
-
+
-Administrative news will be posted here by me. -- Loop
+Administrative news will be posted here by me. -- Loop

groups/local.control.news/2

@@ -1,7 +0,0 @@
-Date: 2024-12-05 07:27:01 GMT-3
-Message-Id: <kocaojivldajgfnjiiou@loop>
-From: Loop
-Subject: account HIMMEL removed by Loop
-Newsgroups: local.control.news
-
-HIMMEL didn't log in a first time (for 1 month) since account creation.

groups/local.control.news/3

@@ -1,7 +0,0 @@
-Date: 2024-12-05 07:52:39 GMT-3
-Message-Id: <tjvhrdqxpqiyixsodahj@loop>
-From: Loop
-Subject: account HIMMEL removed by Loop
-Newsgroups: local.control.news
-
-HIMMEL didn't log in a first time (for 1 month) since account creation.

groups/local.control.news/4

@@ -1,7 +0,0 @@
-Date: 2024-12-05 07:52:39 GMT-3
-Message-Id: <qpxbepgswiifybcnycow@loop>
-From: Loop
-Subject: account MFELIX locked by Loop
-Newsgroups: local.control.news
-
-MFELIX disappeared for over 3 months.

groups/local.control.news/5

@@ -1,7 +0,0 @@
-Date: 2024-12-05 07:52:39 GMT-3
-Message-Id: <ygvnlcmcrzcmtreismjl@loop>
-From: Loop
-Subject: account KIMOCHI locked by Loop
-Newsgroups: local.control.news
-
-KIMOCHI disappeared for over 3 months.

groups/local.control.news/6

@@ -1,7 +0,0 @@
-Date: 2024-12-05 07:52:39 GMT-3
-Message-Id: <zjccjxlroztlxteeoakf@loop>
-From: Loop
-Subject: account WILLIAMP locked by Loop
-Newsgroups: local.control.news
-
-WILLIAMP disappeared for over 3 months.

groups/local.control.news/7

@@ -1,7 +0,0 @@
-Date: 2024-12-05 07:52:39 GMT-3
-Message-Id: <gdcqooyqhkxanqnaxfvc@loop>
-From: Loop
-Subject: account JPMAB locked by Loop
-Newsgroups: local.control.news
-
-JPMAB disappeared for over 3 months.

groups/local.test/1

@@ -1,7 +1,7 @@
-Date: 2024-03-07 21:44:31 GMT-3
+Date: 2024-03-07 21:44:31 GMT-3
-Message-Id: <edjocyeqzqqhnswlbrbo@loop>
+Message-Id: <edjocyeqzqqhnswlbrbo@loop>
-From: Loop
+From: Loop
-Subject: let there be light
+Subject: let there be light
-Newsgroups: local.test
+Newsgroups: local.test
-
+
-A sample group.
+A sample group.

groups/local.test/1.~1~

@@ -1,7 +0,0 @@
-Date: 2024-03-07 21:44:31 GMT-3
-Message-Id: <edjocyeqzqqhnswlbrbo@loop>
-From: Loop
-Subject: let there be light
-Newsgroups: local.control.news
-
-A sample group.

loop.asd

@@ -1,14 +1,7 @@
-;;; -*- Mode: LISP; Syntax: COMMON-LISP; -*-
+;;; -*- Mode: LISP; Syntax: COMMON-LISP; -*-
-(asdf:defsystem :loop
+(asdf:defsystem :loop
-  :version "0.1"
+  :version "0.1"
-  :description "An NNTP server written in Lisp for a circle of friends."
+  :description "An NNTP server for a circle of friends."
-  :depends-on (:lisp-unit 
+  :depends-on (:lisp-unit :str :uiop :cl-ppcre :local-time :clsql-sqlite3 :clingon
-               :str 
+               :filesystem-utils)
-               :uiop 
+  :components ((:file "loop")))
-               :cl-fad 
-               :cl-ppcre 
-               :local-time 
-               :iterate 
-               :clsql-sqlite3)
-  :components ((:file "loop")))
-

loop.nw

@@ -7,7 +7,7 @@
 \usepackage[T1]{fontenc}
 \usepackage[utf8]{inputenc}
 \usepackage{csquotes}
-\usepackage[brazil]{babel}
+\usepackage{babel}
 
 \usepackage{etoolbox}
 \AtBeginEnvironment{quote}{\small}
@@ -16,14 +16,14 @@
 %% \usepackage[backend=biber]{biblatex}
 %% \addbibresource{refs.bib}
 %% \renewcommand{\cite}{\parencite}
-\usepackage[hyperref]{xcolor}
+% \usepackage[hyperref]{xcolor}
 \usepackage[colorlinks=true,citecolor=]{hyperref} % linkcolor=red
 
 \usepackage{amsmath,amsthm,amssymb}
 \allowdisplaybreaks
 \usepackage{lmodern}
 \usepackage{noweb}
-\noweboptions{brazil,longchunks,smallcode}
+\noweboptions{longchunks,smallcode}
 \DeclareMathOperator{\mdc}{mdc}
 \DeclareMathOperator{\gcdext}{gcdext}
 \DeclareMathOperator{\remainder}{remainder}
@@ -85,25 +85,30 @@ way, the conference takes place.
 But \lp\ has nothing to do with e-mail.  \Lp\ uses a communication
 strategy---called a ``protocol''---that is even older than the web
 itself.  The web started out around 1989--1990 and the protocol
-\lp\ uses was conceived in 1979 and implemented in 1980.  The name of
+\lp\ uses begins its history in 1979, the year in which AT\&T released
-the protocol used by \lp\ is NNTP---Network News Transfer Protocol.
+UNIX V7, which was UUCP-ready.  The name UUCP stands for [U]NIX to
-Since e-mail was already daily practice of members of the Internet
+[U]NIX [c]o[p]y. It's both a protocol and a set of programs for
-back then, many things from e-mail were taken by the NNTP designers.
+copying files between UNIX systems.  People begun exchanging messages
-So, an NNTP message looks a lot like an e-mail message and the
+by copying files between machines.  The idea eventually evolved into a
-two---NNTP and SMTP (the protocol used by e-mail)---can often mingle
+protocol called NNTP---Network News Transfer Protocol---, which is the
-seamlessly.  The impression we get from using NNTP is that we're
+protocol used by \lp.  (The protocol used to be called NetNews
-sending e-mail to a certain group of people.  It's as though the
+Transfer Protocol.)  Since e-mail was already daily practice of
-message goes into a collective mailbox and anyone interested in that
+members of the Internet back then, many things from e-mail were taken
-mailbox reads the messages there.  If anyone would like to reply to a
+by the NNTP designers.  So, an NNTP message looks a lot like an e-mail
-message, they do so and, this way, communication flows among the
+message and the two---NNTP and SMTP (the protocol used by
-interested crowd.  If anyone would like to leave the group, nothing is
+e-mail)---can often mingle seamlessly.  The impression we get from
-needed---the person just doesn't go back to read any more messages.
+using NNTP is that we're sending e-mail to a certain group of people.
-Unlike mailing lists, there is no need to formally commit to reading
+It's as though the message goes into a collective mailbox and anyone
-one of these collective mailboxes and no need to formally notify
+interested in that mailbox reads the messages there.  If anyone would
-anyone or any system that you're not interested in that group any
+like to reply to a message, they do so and, this way, communication
-longer.  These collective mailboxes are called ``news groups'' and are
+flows among the interested crowd.  If anyone would like to leave the
-often written as ``newsgroups''.  And the messages posted to these
+group, nothing is needed---the person just doesn't go back to read any
-news groups are called ``articles''.
+more messages.  Unlike mailing lists, there is no need to formally
+commit to reading one of these collective mailboxes and no need to
+formally notify anyone or any system that you're not interested in
+that group any longer.  These collective mailboxes are called ``news
+groups'' and are often written as ``newsgroups''.  And the messages
+posted to these news groups are called ``articles''.
 
 Just like e-mail and the web, network news is an open protocol.
 Anyone could write a program capable of speaking NNTP.  There are many
@@ -255,19 +260,19 @@ the response.  You can get a string version of these numbers:
 
 \begin{verbatim}
 LOOP> (bytes->string (response-data (dispatch (make-request :verb "HELP"))))
-"What's on the menu today?
+"What's on the menu today?
-ARTICLE fetches full articles
+ARTICLE fetches full articles
-AUTHINFO makes me trust you
+AUTHINFO makes me trust you
-BODY fetches an article body
+BODY fetches an article body
-GROUP sets the current group
+GROUP sets the current group
-HEAD fetches article headers
+HEAD fetches article headers
-HELP displays this menu
+HELP displays this menu
-LIST lists all groups
+LIST lists all groups
-MODE handles the mode request from clients
+MODE handles the mode request from clients
-NEXT increments the article pointer
+NEXT increments the article pointer
-POST posts your article
+POST posts your article
-QUIT politely says good-bye
+QUIT politely says good-bye
-XDD displays your state of affairs
+XDD displays your state of affairs
 XOVER fetches the overview database of a group"
 \end{verbatim}
 
@@ -359,6 +364,18 @@ way, they work together.  And \lp\ handles only a text stream, which
 is why it's so easy to connect a keyboard to it and interact with it
 through the command line as illustrated in Section~\ref{sec:design}.
 
+\section{The description of the package}
+
+<<Description>>=
+An NNTP server for a circle of friends.
+@ 
+
+<<Version>>=
+0.1
+@ 
+
+These chunks are used in [[loop.asd]].
+
 \section{The representation of a client}
 
 How do we represent a client?  A client is typically reading a group
@@ -557,19 +574,143 @@ itself---so we can cascade actions based on a user's request.
 (defun request-quit? (r) (and r (string= 'quit (request-verb r))))
 (defun response-quit? (r) (and r (request-quit? (response-request r))))
 
-(defun main ()
+(defun server-start ()
-  (send-banner!)
   (set-up-tables!)
-  (read-accounts!)
+  (send-banner!)
-  (connect-index! "message-id.db")
-  (create-index!)
   (main-loop))
 
+(defun main ()
+  (let ((app (cli/command)))
+    (clingon:run app)))
+
 (defun send-banner! ()
   (send-response! 
    (make-response :code 200 :data "Welcome! Say ``help'' for a menu.")))
 @ %def main main-loop
 
+\section{Command-line parsing}
+
+We're using the clingon library as per Vincent Dardel suggestion in
+``The Common Lisp Cookbook''.  (Thanks, Vincent!)  We begin with
+writing a description of the program and options it understands.  XXX:
+notice I don't know how to support a two-argument option, so I hacked
+a solution away.
+
+<<Command-line parsing>>=
+(defun cli/command ()
+  (clingon:make-command
+   :name "loop"
+   :description "<<Description>>"
+   :version "<<Version>>"
+   :authors '("Circling Skies <loop@antartida.xyz>")
+   :license "GPL v3"
+   :options (cli/options)
+   :handler #'cli/main))
+
+(defun cli/options () 
+  (list 
+   (clingon:make-option
+    :string
+    :description "<username> <invited-by> creates a new account"
+    :long-name "create-account"
+    :key :create-account)
+   (clingon:make-option
+    :string
+    :description "<username> <new-password> changes password"
+    :long-name "change-passwd"
+    :key :change-passwd)
+   (clingon:make-option
+    :flag
+    :description "lists accounts"
+    :short-name #\l
+    :long-name "list-accounts"
+    :key :list-accounts)
+   (clingon:make-option
+    :flag
+    :description "runs NNTP server reading from stdout"
+    :short-name #\s
+    :long-name "server"
+    :key :server)
+   (clingon:make-option
+    :flag
+    :description "run a REPL on port 4006"
+    :short-name #\r
+    :long-name "repl"
+    :key :repl)
+   (clingon:make-option
+    :flag
+    :description "turn on debug logging on stderr"
+    :long-name "logging"
+    :key :logging)))
+@ 
+
+The command-line options form a language.  The user specifies
+everything he wants with flags.  If he wants nothing, for instance, he
+specifies nothing and then nothing happens.  XXX: I'd like to have a
+default action (which would be running the server) that is invoked by
+default if none of the other options would run.  But I don't know how
+to do that yet.
+
+<<Command-line parsing>>=
+(defun cli/list-accounts ()
+  (println (str:join (crlf-string) (list-users))))
+
+(defun cli/create-account (username args)
+  (let ((invited-by (car args)))
+    (cond ((null invited-by)
+           (println "Must specify who invites the new account."))
+          ((get-account username)
+           (println "Username account ``~a'' already exists." username))
+          ((not (get-account invited-by))
+           (println "Invited-by account ``~a'' doesn't exist." invited-by))
+          (t 
+           (multiple-value-bind (okay? pass-or-error) (new-account! username invited-by)
+             (if okay?
+                 (progn (println "Okay, account ``~a'' created with password ``~a''."
+                                 username pass-or-error)
+                        (notify-user-created username))
+                 (println "Sorry, ~a." pass-or-error)))))))
+
+(defun cli/change-passwd (username args)
+  (let* ((random-passwd (random-string 6))
+         (given-passwd (car args))
+         (new-passwd (or given-passwd random-passwd)))
+    (if (not (get-account change-passwd-account))
+        (println "No such account ``~a''." change-passwd-account)
+        (multiple-value-bind (okay? problem) (change-passwd! change-passwd-account new-passwd)
+          (if okay?
+              (println "Okay, account ~a now has password ``~a''." change-passwd-account new-passwd)
+              (println "Sorry, could not change password: ~a." problem))))))
+@ 
+
+Now let's write the main procedure in command-line parsing.
+
+<<Command-line parsing>>=
+(defvar *debug* nil)
+(defun cli/main (cmd)
+  (read-accounts!)
+  (connect-index! "message-id.db")
+  (create-index!)
+  (let ((args (clingon:command-arguments cmd))
+        (repl (clingon:getopt cmd :repl))
+        (server (clingon:getopt cmd :server))
+        (ca (clingon:getopt cmd :create-account))
+        (change-passwd-account (clingon:getopt cmd :change-passwd))
+        (list-accounts (clingon:getopt cmd :list-accounts))
+        (logging (clingon:getopt cmd :logging)))
+    (setf *debug* logging)
+    (when list-accounts
+      (cli/list-accounts))
+    (when ca
+      (cli/create-account ca args))
+    (when change-passwd-account
+      (cli/change-passwd change-passwd-account args))
+    (when repl
+      (stderr "Running a REPL on localhost:4006...~%"))
+    (when server
+      (server-start))))
+@ %def cli/options cli/command
+
 \section{The request dispatching mechanism}
 
 Dispatching requests means consuming one and invoking the correct
@@ -681,11 +822,15 @@ interpreting such bytes.  That's why we call [[read-sequence]] here.
    (read-file-raw (format nil "~a/~a" g i))))
 
 (defun read-file-raw (path)
-  (let* ((size (sb-posix:stat-size (sb-posix:stat path)))
+  (with-open-file 
-         (a (make-array size)))
+      (in path 
-    (with-open-file (in path :element-type '(unsigned-byte 8))
+          :element-type '(unsigned-byte 8)
-      (read-sequence a in)
+          :if-does-not-exist nil)
-      a)))
+    (when in
+      (let* ((size (sb-posix:stat-size (sb-posix:stat path)))
+             (a (make-array size)))
+        (read-sequence a in)
+        a))))
 
 (defun fetch-body (g i)
   (article-body (parse-article (fetch-article g i))))
@@ -728,7 +873,8 @@ Lisp offers me [[labels]], but [[labels]] don't seem so helpful when
 I'm at the REPL.  When I use [[defun]], I'm able to always invoke the
 procedure at the REPL, but that's not so with [[labels]].  I guess the
 use of [[labels]] is when the procedure is so trivial that we have no
-reason to think we're doing to debug it.} %% TODO
+reason to think we're doing to debug it.} XXX: replace menu with
+[[loop]].
 
 <<Help command>>=
 (defun cmd-help (r)
@@ -803,16 +949,10 @@ Syntax error. Say ``authinfo USER /your-name/'' then ``authinfo PASS /your-pass/
 
 \subsection{{\tt CREATE-ACCOUNT}}
 
-We allow authenticated members to invite their friends.
+We allow authenticated members to invite their friends.  Notice that
-
+we're not doing any kind of checking on the username.  XXX: take a
-%% A propósito, estamos removendo a conta {\tt ROOT} de exibição.  O que
+look at how we verify group names match a certain regex and apply the
-%% significa que {\tt ROOT} não nem mesmo se conectar ao \Lp.  Se
+same check here.
-%% desejarmos que {\tt ROOT} se conecte, talvez a gente possa fazer
-%% código especialmente pra gerenciar a conta dele.  Fazemos assim pra
-%% não permitir que usuários tenham qualquer chance de 
-%% 
-%% (remove-if #'(lambda (u) (equal "ROOT" (account-username u)))
-%%                        (read s))
 
 <<Command create-account>>=
 (defun cmd-create-account (r)
@@ -820,7 +960,7 @@ We allow authenticated members to invite their friends.
       (with-n-args 1 r
         (let* ((args (mapcar #'str:upcase (request-args r)))
                (username (car args)))
-          (multiple-value-bind (username pass-or-error) (new-account! username)
+          (multiple-value-bind (username pass-or-error) (new-account! username (client-username *client*))
             (if (not username)
                 (make-response :code 400 :request r
                                :data (fmt "~a. Choose a new name." pass-or-error))
@@ -841,7 +981,7 @@ We allow authenticated members to invite their friends.
       (setq *accounts* (read s))))
   *accounts*)
 
-(defun new-account! (username)
+(defun new-account! (username invited-by)
   (let* ((u (str:upcase username))
          (p (random-string 6))
          (a (make-account :username u
@@ -849,12 +989,12 @@ We allow authenticated members to invite their friends.
                           :creation (get-universal-time)))) 
     (if (get-account u)
         (values nil (fmt "account ~a already exists" u))
-        (let ((c (get-account (client-username *client*))))
+        (progn
-          (push u (account-friends c))
+          (push u (account-friends (get-account invited-by)))
           (push a *accounts*)
           (write-accounts!)
           (values (str:upcase username) p)))))
-@ %def CREATE-ACCOUNT
+@ %def CREATE-ACCOUNT new-account!
 
 Notice that we have a race condition in [[write-accounts]].  What is
 the problem?  Two processes in parallel may ask for the writing of
@@ -1094,12 +1234,13 @@ even cache the overview of the group.) %% TODO
 (defun article-ready? (path)
   (not (temporary-article? path)))
 
+<<Workaround for list-directories and list-files>>
+
 (defun get-articles (g &optional from to)
-  (in-groups ;; We might want to optimize this some day.  Most likely,
+  (in-groups ;; We might want to optimize this some day.  That's a 
-             ;; though, we'll not be using directories. That's a 
              ;; problem to be studied.
    (let ((as (articles->integers
-              (remove-if #'temporary-article? (cl-fad:list-directory g)))))
+              (remove-if #'temporary-article? (loop-list-files (truename g))))))
      (sort (remove-if-not
             #'(lambda (x) (between? x (or from x) (or to x))) 
             as)
@@ -1120,7 +1261,7 @@ even cache the overview of the group.) %% TODO
                      ls)))
 
 (defun list-groups ()
-  (let ((groups (in-groups (cl-fad:list-directory "."))))
+  (let ((groups (in-groups (loop-list-directories (truename ".")))))
     (sort (mapcar #'(lambda (g) (basename (uiop:unix-namestring g))) groups)
           #'string-lessp)))
 
@@ -1139,6 +1280,61 @@ slash, we invoke [[pathname]].  Otherwise, we invoke {\tt
         (file-namestring s))))
 @ %def get-articles group-high-low
 
+To list directories and files, I have been using
+\href{https://github.com/Shinmera/filesystem-utils/tree/master}{{\tt
+    filesystem-utils}} by Yukari Hafner.  I found an issue with both
+     [[list-directories]] and [[list-files]] in a fresh install of
+     FreeBSD 14.2 and in a Debian 8.11 codename jessie.  The issue is
+     that the [[#+cffi]] chunk of the source code incorrectly produced
+     [[NIL]].  (Dramatically, the same was not true in a FreeBSD
+     14.1.)  The source code had an alternative chunk of code for
+     [[#-cffi]] and I discovered that this alternative worked on these
+     systems I tested.  So, as a workaround, I incorporate these
+     procedures below using the chunk [[#-cffi]] to get \Lp\ working
+     on these systems.
+
+<<Workaround for list-directories and list-files>>=
+(defun loop-directory* (directory &rest args &key &allow-other-keys)
+  #+allegro (apply #'directory directory :directories-are-files NIL :follow-symbolic-links NIL args)
+  #+(or clozure digitool) (apply #'directory directory :follow-links NIL args)
+  #+clisp (apply #'directory directory :circle T :if-does-not-exist :ignore args)
+  #+(or cmucl scl) (apply #'directory directory :follow-links NIL :truenamep NIL args)
+  #+lispworks (apply #'directory directory :link-transparency NIL args)
+  #+sbcl (apply #'directory directory :resolve-symlinks NIL args)
+  #-(or allegro clozure digitool clisp cmucl scl lispworks sbcl)
+  (apply #'directory directory args))
+  
+(defun loop-list-files (directory)
+  (let ((directory (pathname-utils:to-directory directory)))
+    (let* ((directory (pathname-utils:pathname* directory))
+           (entries 
+             (ignore-errors
+              (loop-directory* 
+               (merge-pathnames pathname-utils:*wild-file* directory)))))
+      (remove-if #'directory-p entries))))
+
+(defun loop-list-directories (directory)
+  (let ((directory (pathname-utils:to-directory directory)))
+    (let* (#-(or abcl cormanlisp genera xcl)
+           (wild (merge-pathnames
+                  #-(or abcl allegro cmucl lispworks sbcl scl xcl)
+                  pathname-utils:*wild-directory*
+                  #+(or abcl allegro cmucl lispworks sbcl scl xcl) "*.*"
+                  directory))
+           (dirs
+             #+(or abcl xcl) (system:list-directory directory)
+             #+cormanlisp (cl::directory-subdirs directory)
+             #+genera (handler-case (loop for (p . k) in (fs:directory-list directory)
+                                          when (eql :directory k) collect p)
+                        (fs:directory-not-found () nil))
+             #+clozure (ignore-errors (directory* wild :directories T :files NIL))
+             #+mcl (ignore-errors (directory* wild :directories T))
+             #-(or abcl xcl cormanlisp genera clozure mcl) (loop-directory* wild)))
+      (loop for path in dirs
+            when (directory-p path)
+            collect (pathname-utils:force-directory path)))))
+@ %def loop-list-directories loop-list-files
+
 \subsection{{\tt GROUP}}\label{sec:group}
 
 We just need to verify if the group exists and modify [[*client*]].
@@ -1157,10 +1353,10 @@ We just need to verify if the group exists and modify [[*client*]].
 
 (defun group? (g) 
   (in-groups
-   (cl-fad:directory-exists-p g)))
+   (directory-p g)))
 
 (defun xgroup? (g) 
-  (cl-fad:directory-exists-p g))
+  (directory-p g))
 
 (defun set-group! (g)
   (setf (client-group *client*) g))
@@ -1214,27 +1410,39 @@ Section~\ref{sec:index} for the implementation of the index.
   (typical-cmd-response 222 r g i #'(lambda (a) (encode-body (article-body (parse-article a))))))
 @
 
-When we process such commands, it's typical that we need to verify the
+In processing such commands, we need to verify the existence of files
-existence of files {\em et cetera}.  The procedure that might throw
+{\em et cetera}.  The procedure [[fetch-article]] returns [[nil]] when
-[[sb-posix:syscall-error]] is [[sb-posix:stat-size]], which we use to
+the article requested doesn't exist and it throws
-know how many bytes are there in an article, a necessary task in
+[[sb-posix:syscall-error]] due to [[sb-posix:stat-size]], which we
-producing the \verb|OVERVIEW|.
+need to find the size of the article.  We need to know the file size
+not only to allocate an array at the right size, but also because we
+must provide the size when producing the \verb|OVERVIEW|.  If a
+problem such as [[sb-posix:syscall-error]] appears, we just inform the
+client and terminate the request---nothing else to do.  
+
+XXX: instead of only catching [[sb-posix:syscall-error]], we should
+catch anything else, reporting the error.  Otherwise, we will blow up
+in case of some unexpected error, which might not be a bad idea---as
+long as we can log these errors and get a report later on of what's
+going on so we can improve the code.
 
 <<Commands head, body, article>>=
 (defun typical-cmd-response (code r g i get-data)
-  (let ((a (handler-case (fetch-article g i)
+  (handler-case 
-             (sb-posix:syscall-error (c)
+      (let ((a (fetch-article g i)))
-               (make-response :code 400 :request r 
+        (cond ((null a) 
-                              :data (format nil "article ~a/~a: ~a" g i c)))
+               (make-response 
-             (sb-ext:file-does-not-exist (c)
+                :code 400 :request r 
-               (declare (ignore c))
+                :data (format nil "article ~a/~a does not exist" g i)))
-               (make-response :code 400 :request r 
+              (t 
-                              :data (format nil "article ~a/~a does not exist" g i))))))
+               (prepend-response-with
-    (cond ((typep a 'response) a)
+                  (format nil "~a ~a" i (extract-mid a))
-          (t (prepend-response-with
+                  (make-response :multi-line 'yes :code code 
-              (format nil "~a ~a" i (extract-mid a))
+                                 :request r :data (funcall get-data a))))))
-              (make-response :multi-line 'yes :code code 
+    (sb-posix:syscall-error (c)
-                             :request r :data (funcall get-data a)))))))
+      (make-response 
+       :code 400 :request r 
+       :data (format nil "article ~a/~a: ~a" g i c)))))
 @ %def typical-cmd-response
 
 The command \verb|NEXT| has a slight different semantics.
@@ -1506,10 +1714,17 @@ user to either remove the invalid group of type it up properly.
     (setf (account-last-post u) (get-universal-time))))
 @ %def update-last-post-date!
 
-If [[save-article-try]] returns [[NIL]], then [[probe-file]]
+The job of [[save-article-try]] is to atomically allocate an article
-has found an article with name [[name-try]], that is, the procedure
+name in a group.  The atomicity is achieved with [[:if-exists]] in the
-is only successful if [[name-try]] is not yet taken and the writing
+[[with-open-file]] macro.  When [[if:-exists]] is used, the {\tt open}
-takes place successfully.
+system call uses the flag \verb|O_EXCL|, given us an atomic operation.
+So, the first [[with-open-file]] allocates the name.  If successful,
+we take our time writing the article to the temporary file and we
+atomically rename it at the end.  We should close [[name]] before
+trying to [[rename-no-extention]]---on UNIX systems, renaming an open
+target might be okay, but that's not allowed on Windows systems.  Even
+though we have no interest in running \Lp\ on Windows, closing before
+renaming it sounds more like the Right Thing to do.
 
 <<Command post>>=
 (defun rename-no-extension (old new)
@@ -1521,9 +1736,12 @@ takes place successfully.
     (with-open-file 
         (s name 
            :direction :output
-           :if-exists :error ;; an atomic operation
+           :if-exists nil ;; an atomic operation
-           :if-does-not-exist :create))
+           :if-does-not-exist :create)
-    ;(format t "save-article-try: ~a~%" name)
+      (when (null s)
+          (progn
+            (stderr "warning: save-article-try: ~a exists~%" name)
+            (return-from save-article-try 'name-exists))))
     (with-open-file 
         (s tmp
            :direction :output
@@ -1532,42 +1750,25 @@ takes place successfully.
            :element-type '(unsigned-byte 8))
       (write-sequence bs s))
     (rename-no-extension tmp name)))
-@ 
+@ %def save-article-try
 
-The procedure [[save-article-insist]] can return [[NIL]] and still
+The procedure [[save-article-insist]] insists on calling
-have perfectly done its job: it's possible for [[insert-index]] to
+[[save-article-try]] until it finds an article name that has not been
-return [[NIL]] because [[message-id]] may already exist in the index,
+allocated.  Notice that the argument [[name]] is an integer, so
-but that may be no error---for example, when cross-posting.  The
+[[name]] is incremented at each iteration.
-strategy is to write the article using [[name-try]].  If it's not
-possible to write it because of a [[sb-ext:file-exists]] condition,
-then we try the new name {\tt (1+ name-try)} and we repeat these
-attempts until we make it.  If other condition appears, we let it
-propagate up the stack.  If we get to the second [[let]], it's because
-the article has been saved successfully, so we finish with whatever it
-is that [[insert-index-or-log-failure]] must do.
 
 <<Command post>>=
 (defun save-article-insist (g name a message-id)
   (loop for name from name do
     (in-dir (format nil "groups/~a/" g)
-            (handler-case
+            (when (not (eql 'name-exists (save-article-try name a)))
-                (save-article-try name a)
+              (return (values name (insert-index message-id g (fmt "~a" name))))))))
-              (sb-ext:file-exists ()
-                ;; We might want to log the fact.
-                ;(format t "name ~a already exists...~%" name)
-                )
-              (:no-error (new before after) ;; the return values from return-file
-                (declare (ignore new before after))
-                (return (values name (insert-index message-id g (fmt "~a" name)))))))))
-
-(defun get-next-article-name (g)
-  (format nil "~a" (get-next-article-id g)))
 
 (defun get-next-article-id (g)
   (multiple-value-bind (low high len) (group-high-low g)
     (declare (ignore low len))
     (1+ high)))
-@ 
+@ %def save-article-insist get-next-article-id
 
 {\bf How to read lines in the NNTP protocol?}  We've implemented the
 most trivial strategy possible.  It's also the slowest.  What I think
@@ -1860,10 +2061,10 @@ specify anything.
   (clsql:execute-command "create unique index if not exists idx_id_1 
     on indices (id)"))
 
-(defun remake-index! ()
+(defun drop-create-index! ()
   (clsql:execute-command "drop table if exists indices")
   (create-index!))
-@ %def create-index! remake-index! connect-index!
+@ %def create-index! drop-create-index! connect-index!
 
 Of course, the creation and connection of the index must occur before
 [[main-loop]], so it takes place in [[main]].  
@@ -1986,19 +2187,23 @@ we need to index it.
                (insert-index m g i)))))
 
 (defun remake-index-from-fs ()
-  (remake-index!)
+  (drop-create-index!)
   (index-from-fs!))
 @
 
 Here's a program to build the index from a UNIX shell.
 
 <<build-index-from-fs.lisp>>=
-(load "~/.sbclrc")
+#-quicklisp
+(let ((quicklisp-init (merge-pathnames "quicklisp/setup.lisp"
+                                       (user-homedir-pathname))))
+  (when (probe-file quicklisp-init)
+    (load quicklisp-init)))
 (ql:quickload :loop :silent t)
 (in-package #:loop)
 (connect-index! "message-id.db")
-(remake-index!)
+(drop-create-index!)
-(index-from-fs)
+(index-from-fs!)
 (format t "Index built.~%")
 @ 
 
@@ -2019,21 +2224,25 @@ Index built.
 
 \section{Deletion and locking of inactive accounts}\label{sec:inactive-users}
 
-In [[remove-friend]], note that [[username]] is the account name and
+XXX: remove this paragraph from here; present the program first and
-[[friend]] is the name of the account being removed.  Notice as well
+then talk about it.  In [[remove-friend]], note that [[username]] is
-that we only know who invited the person after we can get a hold of
+the account name and [[friend]] is the name of the account being
-the account in [[accounts.lisp]].  This means we must scan each
+removed.  Notice as well that we only know who invited the person
-account to delete an account---we can't delete an account and still
+after we can get a hold of the account in [[accounts.lisp]].  This
-leave the account as someone's friend.
+means we must scan each account to delete an account---we can't delete
+an account and still leave the account as someone's friend.
 
 The program [[cron-remove-inactive-users.lisp]] can be executed every
 day at midnight, say.
 
 <<cron-remove-inactive-users.lisp>>=
-(load "~/.sbclrc")
+#-quicklisp
+(let ((quicklisp-init (merge-pathnames "quicklisp/setup.lisp"
+                                       (user-homedir-pathname))))
+  (when (probe-file quicklisp-init)
+    (load quicklisp-init)))
 (ql:quickload :loop :silent t)
 (in-package #:loop)
-;; (format t *default-pathname-defaults*)
 (read-accounts!)
 (connect-index! "message-id.db")
 (remove-inactive-users!)
@@ -2069,22 +2278,33 @@ with [[setf]]. %% TODO
                             (fmt "disappeared for over ~a months" 
                                  *months-inactive-allowed*))
              (format t "Locked ~a due to long-time-no-see.~%" username))))))
+@ %def remove-inactive-users!
 
+To remove an account, we need to first remove the username (to be
+removed) from anyone's list of friends.  So, this involves scanning
+the entire list of accounts.  Also, notice that delete ``may modify
+{\em sequence}''.  More importantly is to understand tha we really
+must {\tt setf} the return, otherwise we might find the deletion did
+not take effect---for example, when deleting the first element of a
+list.  (This deserves a better explanation, but if you know how linked
+lists are implemented in C, say, then you're likely well aware of how
+it works.)
+
+<<How to remove inactive users>>=
 (defun remove-account! (username)
   (loop for u in *accounts* do
-    (delete-if #'(lambda (x) (equal x username)) (account-friends u)))
+        (setf (account-friends u)
-  (delete-if #'(lambda (x) (equal username (account-username x))) *accounts*))
+              (delete username (account-friends u) :test #'equal)))
+  (setf *accounts*
+        (delete-if #'(lambda (a) (equal (account-username a) username))
+                   *accounts*)))
 
 (defun lock-account! (username why)
   (let ((u (get-account username)))
     (setf (account-pass-locked u) (account-pass u))
     (setf (account-pass u) "locked")
     (setf (account-pass-locked-why u) why)))
-
+@ %def remove-account!
-(defun remove-friend (username friend)
-  (remove-if #'(lambda (x) (equal x friend)) 
-             (account-friends (get-account username))))
-@ %def remove-account! remove-friend
 
 Accounts that do not have a creation date up until today---Tue Sep 17
 21:37:18 ESAST 2024---will have its creation dates migrated to the
@@ -2108,7 +2328,11 @@ New system administrators of \Lp\ will never need to run this.
 Here's a program to run the migration in a UNIX shell.
 
 <<migrate-add-creation-date.lisp>>=
-(load "~/.sbclrc")
+#-quicklisp
+(let ((quicklisp-init (merge-pathnames "quicklisp/setup.lisp"
+                                       (user-homedir-pathname))))
+  (when (probe-file quicklisp-init)
+    (load quicklisp-init)))
 (ql:quickload :loop :silent t)
 (in-package #:loop)
 (migrate-add-creation-and-post-date!)
@@ -2240,15 +2464,22 @@ means 2 bytes.  So our conversion is just removing the first byte.
 (defun plural (v suffix)
   (if (> v 1) "s" ""))
 
-(defun debug? () nil)
-
 (eval-when (:compile-toplevel :load-toplevel :execute)
   (defun fmt (cstr &rest args)
     (apply #'format nil (list* cstr args))))
 
+(defun out (stream &rest args)
+  (apply #'format (cons stream args)))
+
 (defun stderr (&rest args)
-  (when (debug?)
+  (when *debug*
-      (apply #'format (cons *error-output* args))))
+    (apply #'out (cons *error-output* args))))
+
+(defun stdout (&rest args)
+  (apply #'out (list* *standard-output* args)))
+
+(defun println (&rest args)
+  (apply #'out (list* *standard-output* (str:concat (car args) "~%") (cdr args))))
 
 (defun enumerate (ls &optional (first-index 0))
   (loop for e in ls and i from first-index
@@ -2320,18 +2551,28 @@ the code.
 
 \section{How to produce the binary executable}
 
-Just say {\tt make exe} to your shell.
+Just say {\tt make loop} to your shell.
 
 <<build-exe.lisp>>=
-(load "~/.sbclrc")
+#-quicklisp
+(let ((quicklisp-init (merge-pathnames "quicklisp/setup.lisp"
+                                       (user-homedir-pathname))))
+  (when (probe-file quicklisp-init)
+    (load quicklisp-init)))
 (ql:quickload :loop :silent t)
-(sb-ext:save-lisp-and-die #P"loop.exe"
+(sb-ext:save-lisp-and-die #P"loop"
                          :toplevel #'loop:main
-                         :executable t)
+                         :executable t
+                         :save-runtime-options t)
 @
 
 \section{How to update the remote server}\label{sec:live}
 
+XXX: notice we don't include such targets in the [[Makefile]]
+anymore. Now we use a [[Makefile.personal]] that we don't release
+anymore.  We may still keep this section as instruction, but we need
+to update it to reflect the facts.
+
 We automate here the process of updating and compilation of a new
 version of \lp.  It's certain that what we document here is specific
 to a single UNIX system, but what's important is that you (dear
@@ -2417,18 +2658,16 @@ variables in the chunks where it's used.  But for someone reading
 global variables at the top of the file.  That's something to think
 about.
 
+<<List of packages to be loaded>>=
+:lisp-unit :str :uiop :cl-ppcre :local-time :clsql-sqlite3 :clingon
+:filesystem-utils
+@
+
 <<loop.lisp>>=
 ;;; -*- mode: LISP; syntax: COMMON-LISP; package: LOOP; -*-
 (eval-when (:compile-toplevel :load-toplevel :execute)
   (ql:quickload 
-   '(:lisp-unit 
+   '(<<List of packages to be loaded>>)
-     :str 
-     :uiop 
-     :cl-fad 
-     :cl-ppcre 
-     :local-time 
-     :iterate 
-     :clsql-sqlite3)
    :silent t))
 
 (clsql:enable-sql-reader-syntax)
@@ -2436,11 +2675,13 @@ about.
 (defpackage #:loop
   (:use :common-lisp :local-time)
   (:import-from :lisp-unit define-test)
-  (:import-from :iterate iter)
+  (:import-from :org.shirakumo.filesystem-utils 
+                directory-p list-directories list-files)
   (:export :main))
 
 (in-package #:loop)
 
+<<Command-line parsing>>
 <<How to remove inactive users>>
 <<How to enumerate inactive accounts>>
 <<How to migrate accounts without a creation date>>
@@ -2487,37 +2728,58 @@ about.
 <<loop.asd>>=
 ;;; -*- Mode: LISP; Syntax: COMMON-LISP; -*-
 (asdf:defsystem :loop
-  :version "0.0.1"
+  :version "<<Version>>"
-  :description "An NNTP server for a group of friends."
+  :description "<<Description>>"
-  :depends-on (:lisp-unit :str :uiop :cl-fad :cl-ppcre)
+  :depends-on (<<List of packages to be loaded>>)
   :components ((:file "loop")))
-@ %def :loop
+@ %def :loop loop.asd
 
 \section{The UNIX service}
 
+XXX: this section should be informative only.  We're going to suggest
+users how to run the system.  We may provide a [[make install]] target
+that runs things the way we do, but this should be optional.  Idea:
+instead of hard coding a path to the service, make it {\tt conf-home}
+or {\tt conf-service}.
+
+The installation is as follows.  You clone the repo to your
+local-projects, then run make build.  This builds the executable.  You
+edit conf-home to choose your install directory.  Then you say make
+install which copies loop, accounts.lisp, the scripts and the service
+directory.  It is now the syadmin duty to do ln -s ./svc to
+/service/loop, which runs it.  Let's see if we can pull that off.
+
 We use the {\tt tcpserver} program by Daniel J. Bernstein from the
 package \href{https://cr.yp.to/ucspi-tcp.html}{\tt ucspi-tcp}.
 
-<<path to the service>>=
-/home/dbastos/loop
-@ %def
-
 <<port number>>=
 119
 @ %def
 
 <<run>>=
 #!/bin/sh
+home=`head -1 conf-home`
+cd $home
+exec 2>1&
 echo loop
-cd <<path to the service>>
+exec "$home"/tcpserver -HR 0.0.0.0 <<port number>> "$home"/loop
-exec <<path to the service>>/tcpserver -HR 0.0.0.0 <<port number>> <<path to the service>>/loop.exe
 @ %def
 
-How should you update the server if you modified the source-code?  See
+<<log-run>>=
-Section~\ref{sec:live}.
+#!/bin/sh
+echo loop
+exec /usr/bin/logger -i -t loop
+@ 
 
 \section{The writing process}
 
+XXX: note to self.  Targets [[make build]] and [[make install]] must
+be completely non-dependent on noweb.  Also, most users will not run
+any web at all---they'll run noweb, so releasing {\tt any} use in the
+Makefile makes no sense to users.  I think we'll need to set up a
+virtual machine to practice the use of real-world noweb for other
+users.  (Lots of work!)
+
 The program {\tt latexmk} is iseful when I'm writing \LaTeX\ in
 general, but to get the attention of {\tt latexmk} we need to rewrite
 {\tt loop.tex}.  So what I do while writing \lp\ is to have a

peat

@@ -1,224 +0,0 @@
-#!/usr/bin/env python
-# -*- coding: utf8 -*-
-
-                         ##############################
-                        #  ____    ___   ____  ______  #
-                        # |    \  /  _] /    T|      T #
-                        # |  o  )/  [_ Y  o  ||      | #
-                        # |   _/Y    _]|     |l_j  l_j #
-                        # |  |  |   [_ |  _  |  |  |   #
-                        # |  |  |     T|  |  |  |  |   #
-                        # l__j  l_____jl__j__j  l__j   #
-                        #                              #
-                         #####                    #####
-                              # Repeat commands! #
-                               ##################
-
-import errno, os, subprocess, sys, time
-from optparse import OptionParser
-
-
-interval = 1.0
-command = 'true'
-clear = True
-get_paths = lambda: set()
-verbose = True
-dynamic = None
-last_run = None
-
-
-USAGE = r"""usage: %prog [options] COMMAND
-
-COMMAND should be given as a single argument using a shell string.
-
-A list of paths to watch should be piped in on standard input.
-
-For example:
-
-    find . | peat './test.sh'
-    find . -name '*.py' | peat 'rm *.pyc'
-    find . -name '*.py' -print0 | peat -0 'rm *.pyc'
-
-If --dynamic is used, the given command will be run each time to generate the
-list of files to check:
-
-    peat --dynamic 'find .' './test.sh'
-    peat --dynamic 'find . -name '\''*.py'\''' 'rm *.pyc'
-"""
-
-
-def log(s):
-    if verbose:
-        print(s)
-
-def die(s):
-    sys.stderr.write('ERROR: ' + s + '\n')
-    sys.exit(1)
-
-def check(paths):
-    for p in paths:
-        try:
-            if os.stat(p).st_mtime >= last_run:
-                return True
-        except OSError as e:
-            # If the file has been deleted since we started watching, don't
-            # worry about it.
-            if e.errno == errno.ENOENT:
-                pass
-            else:
-                raise
-    return False
-
-def run():
-    global last_run
-    last_run = time.time()
-    log("running: " + command)
-    subprocess.call(command, shell=True)
-
-def build_option_parser():
-    p = OptionParser(USAGE)
-
-    # Main options
-    p.add_option('-i', '--interval', default=None,
-                 help='interval between checks in milliseconds',
-                 metavar='N')
-    p.add_option('-I', '--smart-interval', dest='interval',
-                 action='store_const', const=None,
-                 help='determine the interval based on number of files watched (default)')
-
-    p.add_option('-d', '--dynamic', default=None,
-                 help='run COMMAND before each run to generate the list of files to check',
-                 metavar='COMMAND')
-    p.add_option('-D', '--no-dynamic', dest='dynamic',
-                 action='store_const', const=None,
-                 help='take a list of files to watch on standard in (default)')
-
-    p.add_option('-c', '--clear', default=True,
-                 action='store_true', dest='clear',
-                 help='clear screen before runs (default)')
-    p.add_option('-C', '--no-clear',
-                 action='store_false', dest='clear',
-                 help="don't clear screen before runs")
-
-    p.add_option('-v', '--verbose', default=True,
-                 action='store_true', dest='verbose',
-                 help='show extra logging output (default)')
-    p.add_option('-q', '--quiet',
-                 action='store_false', dest='verbose',
-                 help="don't show extra logging output")
-
-    p.add_option('-w', '--whitespace', default=None,
-                 action='store_const', dest='sep', const=None,
-                 help="assume paths are separated by whitespace (default)")
-    p.add_option('-n', '--newlines',
-                 action='store_const', dest='sep', const='\n',
-                 help="assume paths are separated by newlines")
-    p.add_option('-s', '--spaces',
-                 action='store_const', dest='sep', const=' ',
-                 help="assume paths are separated by spaces")
-    p.add_option('-0', '--zero',
-                 action='store_const', dest='sep', const='\0',
-                 help="assume paths are separated by null bytes")
-
-    return p
-
-
-def _main():
-    if dynamic:
-        log("Running the following command to generate watch list:")
-        log('  ' + dynamic)
-        log('')
-
-    log("Watching the following paths:")
-    for p in get_paths():
-        log('  ' + p)
-    log('')
-    log('Checking for changes every %d milliseconds.' % int(interval * 1000))
-    log('')
-
-    run()
-
-    while True:
-        time.sleep(interval)
-        if check(get_paths()):
-            if clear:
-                subprocess.check_call('clear')
-            run()
-
-def smart_interval(count):
-    """Return the smart interval to use in milliseconds."""
-    if count >= 50:
-        return 1000
-    else:
-        sq = lambda n: n * n
-        return int(1000 * (1 - (sq(50.0 - count) / sq(50))))
-
-def _parse_interval(options):
-    global get_paths
-    if options.interval:
-        i = int(options.interval)
-    elif options.dynamic:
-        i = 1000
-    else:
-        i = smart_interval(len(get_paths()))
-
-    return i / 1000.0
-
-def _parse_paths(sep, data):
-    if not sep:
-        paths = data.split()
-    else:
-        paths = data.split(sep)
-
-    paths = [p.rstrip('\n') for p in paths if p]
-    paths = map(os.path.abspath, paths)
-    paths = set(paths)
-
-    return paths
-
-def main():
-    global interval, command, clear, get_paths, verbose, dynamic
-
-    (options, args) = build_option_parser().parse_args()
-
-    if len(args) != 1:
-        die("exactly one command must be given")
-
-    command = args[0]
-    clear = options.clear
-    verbose = options.verbose
-    sep = options.sep
-    dynamic = options.dynamic
-
-    if dynamic:
-        def _get_paths():
-            data = subprocess.check_output(dynamic, shell=True)
-            return _parse_paths(sep, data)
-
-        get_paths = _get_paths
-    else:
-        data = sys.stdin.read()
-        paths = _parse_paths(sep, data)
-
-        if not paths:
-            die("no paths to watch were given on standard input")
-
-        for path in paths:
-            if not os.path.exists(path):
-                die('path to watch does not exist: ' + repr(path))
-
-        get_paths = lambda: paths
-
-    interval = _parse_interval(options)
-
-    _main()
-
-
-if __name__ == '__main__':
-    import signal
-    def sigint_handler(signal, frame):
-        sys.stdout.write('\n')
-        sys.exit(130)
-    signal.signal(signal.SIGINT, sigint_handler)
-    main()
-

scripts/build-exe.lisp

@@ -0,0 +1,10 @@
+#-quicklisp
+(let ((quicklisp-init (merge-pathnames "quicklisp/setup.lisp"
+                                       (user-homedir-pathname))))
+  (when (probe-file quicklisp-init)
+    (load quicklisp-init)))
+(ql:quickload :loop :silent t)
+(sb-ext:save-lisp-and-die #P"loop"
+                         :toplevel #'loop:main
+                         :executable t
+                         :save-runtime-options t)

scripts/build-index-from-fs.lisp

@@ -0,0 +1,11 @@
+#-quicklisp
+(let ((quicklisp-init (merge-pathnames "quicklisp/setup.lisp"
+                                       (user-homedir-pathname))))
+  (when (probe-file quicklisp-init)
+    (load quicklisp-init)))
+(ql:quickload :loop :silent t)
+(in-package #:loop)
+(connect-index! "message-id.db")
+(drop-create-index!)
+(index-from-fs!)
+(format t "Index built.~%")

scripts/cron-remove-inactive-users.lisp

@@ -0,0 +1,11 @@
+#-quicklisp
+(let ((quicklisp-init (merge-pathnames "quicklisp/setup.lisp"
+                                       (user-homedir-pathname))))
+  (when (probe-file quicklisp-init)
+    (load quicklisp-init)))
+(ql:quickload :loop :silent t)
+(in-package #:loop)
+(read-accounts!)
+(connect-index! "message-id.db")
+(remove-inactive-users!)
+(write-accounts!)

scripts/migrate-add-creation-date.lisp

@@ -0,0 +1,9 @@
+#-quicklisp
+(let ((quicklisp-init (merge-pathnames "quicklisp/setup.lisp"
+                                       (user-homedir-pathname))))
+  (when (probe-file quicklisp-init)
+    (load quicklisp-init)))
+(ql:quickload :loop :silent t)
+(in-package #:loop)
+(migrate-add-creation-and-post-date!)
+(format t "Accounts rewritten.~%")