*** empty log message ***
[gnus] / lisp / nnml.el
index f611fa6..b35c045 100644 (file)
@@ -1,7 +1,7 @@
 ;;; nnml.el --- mail spool access for Gnus
 ;; Copyright (C) 1995 Free Software Foundation, Inc.
 
-;; Author: Lars Ingebrigtsen <larsi@ifi.uio.no>
+;; Author: Lars Magne Ingebrigtsen <larsi@ifi.uio.no>
 ;;     Masanobu UMEDA <umerin@flab.flab.fujitsu.junet>
 ;; Keywords: news, mail
 
@@ -24,6 +24,8 @@
 ;;; Commentary:
 
 ;; Based on nnspool.el by Masanobu UMEDA <umerin@flab.flab.fujitsu.junet>.
+;; For an overview of what the interface functions do, please see the
+;; Gnus sources.  
 
 ;;; Code:
 
@@ -31,7 +33,7 @@
 (require 'nnmail)
 
 (defvar nnml-directory "~/Mail/"
-  "Mail directory.")
+  "Mail spool directory.")
 
 (defvar nnml-active-file (concat nnml-directory "active")
   "Mail active file.")
@@ -51,27 +53,49 @@ the `nnml-generate-nov-databases' command. The function will go
 through all nnml directories and generate nov databases for them
 all. This may very well take some time.")
 
+(defvar nnml-prepare-save-mail-hook nil
+  "Hook run narrowed to an article before saving.")
+
 \f
 
-(defconst nnml-version "nnml 0.2"
+(defconst nnml-version "nnml 1.0"
   "nnml version.")
 
-(defvar nnml-current-directory nil
-  "Current news group directory.")
+(defvar nnml-nov-file-name ".overview")
 
+(defvar nnml-current-directory nil)
+(defvar nnml-current-group nil)
 (defvar nnml-status-string "")
-
 (defvar nnml-nov-buffer-alist nil)
-
 (defvar nnml-group-alist nil)
+(defvar nnml-active-timestamp nil)
+
+\f
+
+;; Server variables.
+
+(defvar nnml-current-server nil)
+(defvar nnml-server-alist nil)
+(defvar nnml-server-variables 
+  (list 
+   (list 'nnml-directory nnml-directory)
+   (list 'nnml-active-file nnml-active-file)
+   (list 'nnml-newsgroups-file nnml-newsgroups-file)
+   (list 'nnml-get-new-mail nnml-get-new-mail)
+   (list 'nnml-nov-is-evil nnml-nov-is-evil)
+   (list 'nnml-nov-file-name nnml-nov-file-name)
+   '(nnml-current-directory nil)
+   '(nnml-current-group nil)
+   '(nnml-status-string "")
+   '(nnml-nov-buffer-alist nil)
+   '(nnml-group-alist nil)
+   '(nnml-active-timestamp nil)))
 
 \f
 
 ;;; Interface functions.
 
-(defun nnml-retrieve-headers (sequence &optional newsgroup server)
-  "Retrieve the headers for the articles in SEQUENCE.
-Newsgroup must be selected before calling this function."
+(defun nnml-retrieve-headers (sequence &optional newsgroup server fetch-old)
   (save-excursion
     (set-buffer nntp-server-buffer)
     (erase-buffer)
@@ -79,151 +103,214 @@ Newsgroup must be selected before calling this function."
          (number (length sequence))
          (count 0)
          beg article)
-      (nnml-possibly-change-directory newsgroup)
-      (if (nnml-retrieve-headers-with-nov sequence)
-         'nov
-       (while sequence
-         (setq article (car sequence))
-         (setq file
-               (concat nnml-current-directory (prin1-to-string article)))
-         (if (and (file-exists-p file)
-                  (not (file-directory-p file)))
-             (progn
-               (insert (format "221 %d Article retrieved.\n" article))
-               (setq beg (point))
-               (insert-file-contents file)
-               (goto-char beg)
-               (if (search-forward "\n\n" nil t)
-                   (forward-char -1)
-                 (goto-char (point-max))
-                 (insert "\n\n"))
-               (insert ".\n")
-               (delete-region (point) (point-max))))
-         (setq sequence (cdr sequence))
-         (setq count (1+ count))
+      (if (stringp (car sequence))
+         'headers
+       (nnml-possibly-change-directory newsgroup)
+       (if (nnml-retrieve-headers-with-nov sequence fetch-old)
+           'nov
+         (while sequence
+           (setq article (car sequence))
+           (setq file
+                 (concat nnml-current-directory (int-to-string article)))
+           (if (and (file-exists-p file)
+                    (not (file-directory-p file)))
+               (progn
+                 (insert (format "221 %d Article retrieved.\n" article))
+                 (setq beg (point))
+                 (nnheader-insert-head file)
+                 (goto-char beg)
+                 (if (search-forward "\n\n" nil t)
+                     (forward-char -1)
+                   (goto-char (point-max))
+                   (insert "\n\n"))
+                 (insert ".\n")
+                 (delete-region (point) (point-max))))
+           (setq sequence (cdr sequence))
+           (setq count (1+ count))
+           (and (numberp nnmail-large-newsgroup)
+                (> number nnmail-large-newsgroup)
+                (zerop (% count 20))
+                gnus-verbose-backends
+                (message "nnml: Receiving headers... %d%%"
+                         (/ (* count 100) number))))
+
          (and (numberp nnmail-large-newsgroup)
               (> number nnmail-large-newsgroup)
-              (zerop (% count 20))
               gnus-verbose-backends
-              (message "nnml: Receiving headers... %d%%"
-                       (/ (* count 100) number))))
-
-       (and (numberp nnmail-large-newsgroup)
-            (> number nnmail-large-newsgroup)
-            gnus-verbose-backends
-            (message "nnml: Receiving headers... done"))
-
-       ;; Fold continuation lines.
-       (goto-char 1)
-       (while (re-search-forward "\\(\r?\n[ \t]+\\)+" nil t)
-         (replace-match " " t t))
-       'headers))))
+              (message "nnml: Receiving headers...done"))
 
-(defun nnml-open-server (host &optional service)
-  (setq nnml-status-string "")
-  (nnheader-init-server-buffer))
+         ;; Fold continuation lines.
+         (goto-char (point-min))
+         (while (re-search-forward "\\(\r?\n[ \t]+\\)+" nil t)
+           (replace-match " " t t))
+         'headers)))))
+
+(defun nnml-open-server (server &optional defs)
+  (nnheader-init-server-buffer)
+  (if (equal server nnml-current-server)
+      t
+    (if nnml-current-server
+       (setq nnml-server-alist 
+             (cons (list nnml-current-server
+                         (nnheader-save-variables nnml-server-variables))
+                   nnml-server-alist)))
+    (let ((state (assoc server nnml-server-alist)))
+      (if state 
+         (progn
+           (nnheader-restore-variables (nth 1 state))
+           (setq nnml-server-alist (delq state nnml-server-alist)))
+       (nnheader-set-init-variables nnml-server-variables defs)))
+    (setq nnml-current-server server)))
 
 (defun nnml-close-server (&optional server)
-  "Close news server."
   t)
 
 (defun nnml-server-opened (&optional server)
-  "Return server process status, T or NIL.
-If the stream is opened, return T, otherwise return NIL."
-  (and nntp-server-buffer
-       (get-buffer nntp-server-buffer)))
+  (and (equal server nnml-current-server)
+       nntp-server-buffer
+       (buffer-name nntp-server-buffer)))
 
-(defun nnml-status-message ()
-  "Return server status response as string."
+(defun nnml-status-message (&optional server)
   nnml-status-string)
 
 (defun nnml-request-article (id &optional newsgroup server buffer)
-  "Select article by message ID (or number)."
   (nnml-possibly-change-directory newsgroup)
-  (let ((file (if (stringp id)
-                 nil
-               (concat nnml-current-directory (prin1-to-string id))))
-       (nntp-server-buffer (or buffer nntp-server-buffer)))
-    (if (and (stringp file)
-            (file-exists-p file)
-            (not (file-directory-p file)))
-       (save-excursion
-         (nnmail-find-file file)))))
+  (let* ((group-num (and (stringp id) (nnml-find-group-number id)))
+        (number (if (numberp id) id (cdr group-num)))
+        (file
+         (and number
+              (concat 
+               (if (numberp id)
+                   nnml-current-directory
+                 (nnmail-article-pathname (car group-num) nnml-directory))
+               (int-to-string number))))
+        (nntp-server-buffer (or buffer nntp-server-buffer)))
+    (and file
+        (file-exists-p file)
+        (not (file-directory-p file))
+        (save-excursion (nnmail-find-file file))
+        ;; We return the article number.
+        (cons newsgroup (string-to-int (file-name-nondirectory file))))))
 
 (defun nnml-request-group (group &optional server dont-check)
-  "Select news GROUP."
-  (nnml-possibly-change-directory group)
-  (or dont-check (nnml-get-new-mail))
-  (or nnml-group-alist 
+  (if (not (nnml-possibly-change-directory group))
       (progn
-       (nnml-request-list)
-       (setq nnml-group-alist (nnmail-get-active))))
-  (let ((active (nth 1 (assoc group nnml-group-alist))))
-    (save-excursion
-      (set-buffer nntp-server-buffer)
-      (erase-buffer)
-      (if (not active)
-         ()
-       (insert (format "211 %d %d %d %s\n" 
-                       (max (1+ (- (cdr active) (car active))) 0)
-                       (car active) (cdr active) group))
-       t))))
+       (setq nnml-status-string "Invalid group (no such directory)")
+       nil)
+    (if dont-check 
+       t
+      (nnmail-activate 'nnml)
+      (let ((active (nth 1 (assoc group nnml-group-alist))))
+       (save-excursion
+         (set-buffer nntp-server-buffer)
+         (erase-buffer)
+         (if (not active)
+             ()
+           (insert (format "211 %d %d %d %s\n" 
+                           (max (1+ (- (cdr active) (car active))) 0)
+                           (car active) (cdr active) group))
+           t))))))
+
+(defun nnml-request-scan (&optional group server)
+  (nnmail-get-new-mail 'nnml 'nnml-save-nov nnml-directory group))
 
 (defun nnml-close-group (group &optional server)
   t)
 
+(defun nnml-request-close ()
+  (setq nnml-current-server nil)
+  (setq nnml-server-alist nil)
+  t)
+
+(defun nnml-request-create-group (group &optional server) 
+  (nnmail-activate 'nnml)
+  (or (assoc group nnml-group-alist)
+      (let (active)
+       (setq nnml-group-alist (cons (list group (setq active (cons 1 0)))
+                                    nnml-group-alist))
+       (nnml-possibly-create-directory group)
+       (nnml-possibly-change-directory group)
+       (let ((articles (mapcar
+                        (lambda (file)
+                          (string-to-int file))
+                        (directory-files 
+                         nnml-current-directory nil "^[0-9]+$"))))
+         (and articles
+              (progn
+                (setcar active (apply 'min articles))
+                (setcdr active (apply 'max articles)))))
+       (nnmail-save-active nnml-group-alist nnml-active-file)))
+  t)
+
 (defun nnml-request-list (&optional server)
-  "List active newsgoups."
-  (if server (nnml-get-new-mail))
   (save-excursion
-    (nnmail-find-file nnml-active-file)))
+    (nnmail-find-file nnml-active-file)
+    (setq nnml-group-alist (nnmail-get-active))))
 
 (defun nnml-request-newgroups (date &optional server)
-  "List groups created after DATE."
   (nnml-request-list server))
 
 (defun nnml-request-list-newsgroups (&optional server)
-  "List newsgroups (defined in NNTP2)."
   (save-excursion
     (nnmail-find-file nnml-newsgroups-file)))
 
 (defun nnml-request-post (&optional server)
-  "Post a new news in current buffer."
   (mail-send-and-exit nil))
 
-(fset 'nnml-request-post-buffer 'nnmail-request-post-buffer)
-
 (defun nnml-request-expire-articles (articles newsgroup &optional server force)
-  "Expire all articles in the ARTICLES list in group GROUP.
-The list of unexpired articles will be returned (ie. all articles that
-were too fresh to be expired).
-If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
   (nnml-possibly-change-directory newsgroup)
   (let* ((days (or (and nnmail-expiry-wait-function
                        (funcall nnmail-expiry-wait-function newsgroup))
                   nnmail-expiry-wait))
+        (active-articles 
+         (mapcar
+          (function
+           (lambda (name)
+             (string-to-int name)))
+          (directory-files nnml-current-directory nil "^[0-9]+$" t)))
+        (max-article (and active-articles (apply 'max active-articles)))
+        (is-old t)
         article rest mod-time)
-    (while articles
-      (setq article (concat nnml-current-directory (int-to-string
-                                                     (car articles))))
+    (nnmail-activate 'nnml)
+
+    (while (and articles is-old)
+      (setq article (concat nnml-current-directory 
+                           (int-to-string (car articles))))
       (if (setq mod-time (nth 5 (file-attributes article)))
-         (if (or force
-                 (> (nnmail-days-between
-                     (current-time-string)
-                     (current-time-string mod-time))
-                    days))
+         (if (and (or (not nnmail-keep-last-article)
+                      (not max-article)
+                      (not (= (car articles) max-article)))
+                  (or force
+                      (and (not (equal mod-time '(0 0)))
+                           (setq is-old
+                                 (> (nnmail-days-between
+                                     (current-time-string)
+                                     (current-time-string mod-time))
+                                    days)))))
              (progn
-               (and gnus-verbose-backends (message "Deleting %s..." article))
+               (and gnus-verbose-backends 
+                    (message "Deleting article %s in %s..."
+                             article newsgroup))
                (condition-case ()
-                   (delete-file article)
-                 (file-error nil))
+                   (funcall nnmail-delete-file-function article)
+                 (file-error
+                  (setq rest (cons (car articles) rest))))
+               (setq active-articles (delq (car articles) active-articles))
                (nnml-nov-delete-article newsgroup (car articles)))
            (setq rest (cons (car articles) rest))))
       (setq articles (cdr articles)))
+    (let ((active (nth 1 (assoc newsgroup nnml-group-alist))))
+      (and active
+          (setcar active (or (and active-articles
+                                  (apply 'min active-articles))
+                             0)))
+      (nnmail-save-active nnml-group-alist nnml-active-file))
     (nnml-save-nov)
-    rest))
+    (message "")
+    (nconc rest articles)))
 
-(defun nnml-request-move-article (article group server accept-form)
+(defun nnml-request-move-article 
+  (article group server accept-form &optional last)
   (let ((buf (get-buffer-create " *nnml move*"))
        result)
     (and 
@@ -236,39 +323,39 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
        result)
      (progn
        (condition-case ()
-          (delete-file (concat nnml-current-directory 
-                               (int-to-string article)))
+          (funcall nnmail-delete-file-function
+                   (concat nnml-current-directory 
+                           (int-to-string article)))
         (file-error nil))
        (nnml-nov-delete-article group article)
-       (nnml-save-nov)))
+       (and last (nnml-save-nov))))
     result))
 
-(defun nnml-request-accept-article (group)
+(defun nnml-request-accept-article (group &optional last)
   (let (result)
     (if (stringp group)
        (and 
-        (nnml-request-list)
-        (setq nnml-group-alist (nnmail-get-active))
+        (nnmail-activate 'nnml)
         ;; We trick the choosing function into believing that only one
         ;; group is availiable.  
         (let ((nnmail-split-methods (list (list group ""))))
           (setq result (car (nnml-save-mail))))
         (progn
           (nnmail-save-active nnml-group-alist nnml-active-file)
-          (nnml-save-nov)))
+          (and last (nnml-save-nov))))
       (and
-       (nnml-request-list)
-       (setq nnml-group-alist (nnmail-get-active))
+       (nnmail-activate 'nnml)
        (setq result (car (nnml-save-mail)))
        (progn
         (nnmail-save-active nnml-group-alist nnml-active-file)
-        (nnml-save-nov))))
+        (and last (nnml-save-nov)))))
     result))
 
 (defun nnml-request-replace-article (article group buffer)
   (nnml-possibly-change-directory group)
   (save-excursion
     (set-buffer buffer)
+    (nnml-possibly-create-directory group)
     (if (not (condition-case ()
                 (progn
                   (write-region (point-min) (point-max)
@@ -282,75 +369,182 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
            (art (concat (int-to-string article) "\t"))
            nov-line)
        (setq nov-line (nnml-make-nov-line chars))
+       ;; Replace the NOV line in the NOV file.
        (save-excursion 
          (set-buffer (nnml-open-nov group))
          (goto-char (point-min))
          (if (or (looking-at art)
-                 (search-forward (concat "\n" art)))
-             (progn
-               (delete-region (progn (beginning-of-line) (point))
-                              (progn (forward-line 1) (point)))
-               (insert (int-to-string article) nov-line)
-               (nnml-save-nov))
-           (kill-buffer (current-buffer)))
+                 (search-forward (concat "\n" art) nil t))
+             ;; Delete the old NOV line.
+             (delete-region (progn (beginning-of-line) (point))
+                            (progn (forward-line 1) (point)))
+           ;; The line isn't here, so we have to find out where
+           ;; we should insert it. (This situation should never
+           ;; occur, but one likes to make sure...)
+           (while (and (looking-at "[0-9]+\t")
+                       (< (string-to-int 
+                           (buffer-substring 
+                            (match-beginning 0) (match-end 0)))
+                          article)
+                       (zerop (forward-line 1)))))
+         (beginning-of-line)
+         (insert (int-to-string article) nov-line)
+         (nnml-save-nov)
          t)))))
 
+(defun nnml-request-delete-group (group &optional force server)
+  (nnml-possibly-change-directory group)
+  ;; Delete all articles in GROUP.
+  (if (not force)
+      ()                               ; Don't delete the articles.
+    (let ((articles 
+          (directory-files 
+           nnml-current-directory t
+           (concat "^[0-9]+$\\|" (regexp-quote nnml-nov-file-name) "$"))))
+      (while articles 
+       (and (file-writable-p (car articles))
+            (progn
+              (and gnus-verbose-backends
+                   (message (message "Deleting article %s in %s..."
+                                     (car articles) group)))
+              (funcall nnmail-delete-file-function (car articles))))
+       (setq articles (cdr articles))))
+    ;; Try to delete the directory itself.
+    (condition-case ()
+       (delete-directory nnml-current-directory)
+      (error nil)))
+  ;; Remove the group from all structures.
+  (setq nnml-group-alist 
+       (delq (assoc group nnml-group-alist) nnml-group-alist)
+       nnml-current-group nil
+       nnml-current-directory nil)
+  ;; Save the active file.
+  (nnmail-save-active nnml-group-alist nnml-active-file)
+  t)
+
+(defun nnml-request-rename-group (group new-name &optional server)
+  (nnml-possibly-change-directory group)
+  ;; Rename directory.
+  (and (file-writable-p nnml-current-directory)
+       (condition-case ()
+          (progn
+            (rename-file 
+             (directory-file-name nnml-current-directory)
+             (directory-file-name 
+              (nnmail-article-pathname new-name nnml-directory)))
+            t)
+        (error nil))
+       ;; That went ok, so we change the internal structures.
+       (let ((entry (assoc group nnml-group-alist)))
+        (and entry (setcar entry new-name))
+        (setq nnml-current-directory nil
+              nnml-current-group nil)
+        ;; Save the new group alist.
+        (nnmail-save-active nnml-group-alist nnml-active-file)
+        t)))
 
 \f
-;;; Internal functions
+;;; Internal functions.
 
-(defun nnml-retrieve-headers-with-nov (articles)
+;; Find an article number in the current group given the Message-ID. 
+(defun nnml-find-group-number (id)
+  (save-excursion
+    (set-buffer (get-buffer-create " *nnml id*"))
+    (buffer-disable-undo (current-buffer))
+    (let ((alist nnml-group-alist)
+         number)
+      ;; We want to look through all .overview files, but we want to
+      ;; start with the one in the current directory.  It seems most
+      ;; likely that the article we are looking for is in that group. 
+      (if (setq number (nnml-find-id nnml-current-group id))
+         (cons nnml-current-group number)
+       ;; It wasn't there, so we look through the other groups as well.
+       (while (and (not number)
+                   alist)
+         (or (string= (car (car alist)) nnml-current-group)
+             (setq number (nnml-find-id (car (car alist)) id)))
+         (or number
+             (setq alist (cdr alist))))
+       (and number
+            (cons (car (car alist)) number))))))
+
+(defun nnml-find-id (group id)
+  (erase-buffer)
+  (insert-file-contents 
+   (concat (nnmail-article-pathname group nnml-directory)
+          nnml-nov-file-name))
+  (let (number found)
+    (while (and (not found) 
+               (search-forward id nil t)) ; We find the ID.
+      ;; And the id is in the fourth field.
+      (if (search-backward 
+          "\t" (save-excursion (beginning-of-line) (point)) t 4)
+         (progn
+           (beginning-of-line)
+           (setq found t)
+           ;; We return the article number.
+           (setq number
+                 (condition-case ()
+                     (read (current-buffer))
+                   (error nil))))))
+    number))
+      
+
+(defun nnml-retrieve-headers-with-nov (articles &optional fetch-old)
   (if (or gnus-nov-is-evil nnml-nov-is-evil)
       nil
     (let ((first (car articles))
          (last (progn (while (cdr articles) (setq articles (cdr articles)))
                       (car articles)))
-         (nov (concat nnml-current-directory ".nov")))
+         (nov (concat nnml-current-directory nnml-nov-file-name)))
       (if (file-exists-p nov)
          (save-excursion
            (set-buffer nntp-server-buffer)
            (erase-buffer)
            (insert-file-contents nov)
-           (goto-char 1)
-           (while (and (not (eobp)) (< first (read (current-buffer))))
-             (forward-line 1))
-           (beginning-of-line)
-           (if (not (eobp)) (delete-region 1 (point)))
-           (while (and (not (eobp)) (>= last (read (current-buffer))))
-             (forward-line 1))
-           (beginning-of-line)
-           (if (not (eobp)) (delete-region (point) (point-max)))
-           t)))))
-
-(defun nnml-possibly-change-directory (newsgroup)
+           (if (and fetch-old
+                    (not (numberp fetch-old)))
+               t                       ; Don't remove anything.
+             (if fetch-old
+                 (setq first (max 1 (- first fetch-old))))
+             (goto-char (point-min))
+             (while (and (not (eobp)) (< first (read (current-buffer))))
+               (forward-line 1))
+             (beginning-of-line)
+             (if (not (eobp)) (delete-region 1 (point)))
+             (while (and (not (eobp)) (>= last (read (current-buffer))))
+               (forward-line 1))
+             (beginning-of-line)
+             (if (not (eobp)) (delete-region (point) (point-max)))
+             t))))))
+
+(defun nnml-possibly-change-directory (newsgroup &optional force)
   (if newsgroup
       (let ((pathname (nnmail-article-pathname newsgroup nnml-directory)))
-       (if (file-directory-p pathname)
-           (setq nnml-current-directory pathname)
-         (error "No such newsgroup: %s" newsgroup)))))
-
-(defun nnml-create-directories ()
-  (let ((methods nnmail-split-methods)
-       dir dirs)
-    (while methods
-      (setq dir (nnmail-article-pathname (car (car methods)) nnml-directory))
-      (while (not (file-directory-p dir))
-       (setq dirs (cons dir dirs))
-       (setq dir (file-name-directory (directory-file-name dir))))
-      (while dirs
-       (if (make-directory (directory-file-name (car dirs)))
-           (error "Could not create directory %s" (car dirs)))
-       (and gnus-verbose-backends 
-            (message "Creating mail directory %s" (car dirs)))
-       (setq dirs (cdr dirs)))
-      (setq methods (cdr methods)))))
+       (and (or force (file-directory-p pathname))
+            (setq nnml-current-directory pathname
+                  nnml-current-group newsgroup)))
+    t))
 
+(defun nnml-possibly-create-directory (group)
+  (let (dir dirs)
+    (setq dir (nnmail-article-pathname group nnml-directory))
+    (while (not (file-directory-p dir))
+      (setq dirs (cons dir dirs))
+      (setq dir (file-name-directory (directory-file-name dir))))
+    (while dirs
+      (make-directory (directory-file-name (car dirs)))
+      (and gnus-verbose-backends 
+          (message "Creating mail directory %s" (car dirs)))
+      (setq dirs (cdr dirs)))))
+            
 (defun nnml-save-mail ()
   "Called narrowed to an article."
   (let ((group-art (nreverse (nnmail-article-group 'nnml-active-number)))
        chars nov-line)
     (setq chars (nnmail-insert-lines))
     (nnmail-insert-xref group-art)
+    (run-hooks 'nnml-prepare-save-mail-hook)
     (goto-char (point-min))
     (while (looking-at "From ")
       (replace-match "X-From-Line: ")
@@ -359,6 +553,7 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
     (let ((ga group-art)
          first)
       (while ga
+       (nnml-possibly-create-directory (car (car ga)))
        (let ((file (concat (nnmail-article-pathname 
                             (car (car ga)) nnml-directory)
                            (int-to-string (cdr (car ga))))))
@@ -384,44 +579,19 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
 (defun nnml-active-number (group)
   "Compute the next article number in GROUP."
   (let ((active (car (cdr (assoc group nnml-group-alist)))))
+    ;; The group wasn't known to nnml, so we just create an active
+    ;; entry for it.   
+    (or active
+       (progn
+         (setq active (cons 1 0))
+         (setq nnml-group-alist (cons (list group active) nnml-group-alist))))
     (setcdr active (1+ (cdr active)))
-    (let (file)
-      (while (file-exists-p
-             (setq file (concat (nnmail-article-pathname 
-                                 group nnml-directory)
-                                (int-to-string (cdr active)))))
-       (setcdr active (1+ (cdr active)))))
+    (while (file-exists-p
+           (concat (nnmail-article-pathname group nnml-directory)
+                   (int-to-string (cdr active))))
+      (setcdr active (1+ (cdr active))))
     (cdr active)))
 
-(defun nnml-get-new-mail ()
-  "Read new incoming mail."
-  (let (incoming)
-    (nnml-create-directories)
-    (if (and nnml-get-new-mail nnmail-spool-file
-            (file-exists-p nnmail-spool-file)
-            (> (nth 7 (file-attributes nnmail-spool-file)) 0))
-       (progn
-         (and gnus-verbose-backends 
-              (message "nnml: Reading incoming mail..."))
-         (setq incoming 
-               (nnmail-move-inbox nnmail-spool-file 
-                                  (concat nnml-directory "Incoming")))
-         (nnml-request-list)
-         (setq nnml-group-alist (nnmail-get-active))
-         (nnmail-split-incoming incoming 'nnml-save-mail)
-         (nnmail-save-active nnml-group-alist nnml-active-file)
-         (nnml-save-nov)
-         (run-hooks 'nnmail-read-incoming-hook)
-         ;; The following has been commented away, just to make sure
-         ;; that nobody ever loses any mail. If you feel safe that
-         ;; nnml will never do anything strange, just remove those
-         ;; two semicolons, and avoid having lots of "Incoming*"
-         ;; files. 
-;;         (delete-file incoming)
-         (and gnus-verbose-backends
-              (message "nnml: Reading incoming mail...done"))))))
-
-
 (defun nnml-add-nov (group article line)
   "Add a nov line for the GROUP base."
   (save-excursion 
@@ -483,14 +653,18 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
        (format "\t%s\t%s\t%s\t%s\t%s\t%d\t%s\t%s\t\n"
                (or subject "(none)")
                (or from "(nobody)") (or date "")
-               (or id "") (or references "")
+               (or id (concat "nnml-dummy-id-" 
+                              (mapconcat 
+                               (lambda (time) (int-to-string time))
+                               (current-time) "-")))
+               (or references "")
                (or chars 0) (or lines "0") (or xref ""))))))
 
 (defun nnml-open-nov (group)
   (or (cdr (assoc group nnml-nov-buffer-alist))
       (let ((buffer (find-file-noselect 
                     (concat (nnmail-article-pathname 
-                             group nnml-directory) ".nov"))))
+                             group nnml-directory) nnml-nov-file-name))))
        (save-excursion
          (set-buffer buffer)
          (buffer-disable-undo (current-buffer)))
@@ -504,18 +678,21 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
       (if (buffer-name (cdr (car nnml-nov-buffer-alist)))
          (progn
            (set-buffer (cdr (car nnml-nov-buffer-alist)))
-           (write-region 1 (point-max) (buffer-file-name) nil 'nomesg)
+           (and (buffer-modified-p)
+                (write-region 
+                 1 (point-max) (buffer-file-name) nil 'nomesg))
            (set-buffer-modified-p nil)
            (kill-buffer (current-buffer))))
       (setq nnml-nov-buffer-alist (cdr nnml-nov-buffer-alist)))))
 
+;;;###autoload
 (defun nnml-generate-nov-databases (dir)
   "Generate nov databases in all nnml mail newsgroups."
   (interactive 
    (progn   
      (setq nnml-group-alist nil)
      (list nnml-directory)))
-  (nnml-open-server (system-name))
+  (nnml-open-server (or nnml-current-server ""))
   (let ((dirs (directory-files dir t nil t)))
     (while dirs 
       (if (and (not (string-match "/\\.\\.$" (car dirs)))
@@ -530,7 +707,7 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
                    (string-to-int name)))
                 (directory-files dir nil "^[0-9]+$" t))
                (function <)))
-       (nov (concat dir "/.nov"))
+       (nov (concat dir "/" nnml-nov-file-name))
        (nov-buffer (get-buffer-create "*nov*"))
        nov-line chars)
     (if files
@@ -549,7 +726,7 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
        (save-excursion
          (set-buffer nntp-server-buffer)
          (if (file-exists-p nov)
-             (delete-file nov))
+             (funcall nnmail-delete-file-function nov))
          (save-excursion
            (set-buffer nov-buffer)
            (buffer-disable-undo (current-buffer))
@@ -557,16 +734,18 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
          (while files
            (erase-buffer)
            (insert-file-contents (concat dir "/" (int-to-string (car files))))
-           (goto-char 1)
+           (goto-char (point-min))
            (narrow-to-region 1 (save-excursion (search-forward "\n\n" nil t)
                                                (setq chars (- (point-max) 
                                                               (point)))
                                                (point)))
-           (setq nov-line (nnml-make-nov-line chars))
-           (save-excursion
-             (set-buffer nov-buffer)
-             (goto-char (point-max))
-             (insert (int-to-string (car files)) nov-line))
+           (if (not (= 0 chars))       ; none of them empty files...
+               (progn
+                 (setq nov-line (nnml-make-nov-line chars))
+                 (save-excursion
+                   (set-buffer nov-buffer)
+                   (goto-char (point-max))
+                   (insert (int-to-string (car files)) nov-line))))
            (widen)
            (setq files (cdr files)))
          (save-excursion
@@ -579,8 +758,8 @@ If FORCE is non-nil, ARTICLES will be deleted whether they are old or not."
 (defun nnml-nov-delete-article (group article)
   (save-excursion
     (set-buffer (nnml-open-nov group))
-    (goto-char 1)
-    (if (re-search-forward (concat "^" (int-to-string article) "\t"))
+    (goto-char (point-min))
+    (if (re-search-forward (concat "^" (int-to-string article) "\t") nil t)
        (delete-region (match-beginning 0) (progn (forward-line 1) (point))))
     t))