*** empty log message ***
[gnus] / lisp / nnmail.el
index 8d13ab9..30467f4 100644 (file)
 (require 'nnheader)
 (require 'timezone)
 (require 'message)
-(eval-when-compile (require 'cl))
+(require 'cl)
 (require 'custom)
 
+(eval-and-compile
+  (autoload 'gnus-error "gnus-util"))
+
 (defgroup nnmail nil
   "Reading mail with Gnus."
   :group 'gnus)
@@ -109,7 +112,7 @@ If nil, the first match found will be used."
 
 ;; Added by gord@enci.ucalgary.ca (Gordon Matzigkeit).
 (defcustom nnmail-keep-last-article nil
-  "If non-nil, nnmail will never delete the last expired article in a directory.  
+  "If non-nil, nnmail will never delete the last expired article in a directory.
 You may need to set this variable if other programs are putting
 new mail into folder numbers that Gnus has marked as expired."
   :group 'nnmail-procmail
@@ -157,7 +160,7 @@ Eg.:
   :type '(choice (const :tag "nnmail-expiry-wait" nil)
                 (function :format "%v" nnmail-)))
 
-(defcustom nnmail-spool-file 
+(defcustom nnmail-spool-file
   (or (getenv "MAIL")
       (concat "/usr/spool/mail/" (user-login-name)))
   "Where the mail backends will look for incoming mail.
@@ -230,7 +233,7 @@ to be moved to."
   :group 'nnmail-retrieve
   :type 'boolean)
 
-(defcustom nnmail-read-incoming-hook 
+(defcustom nnmail-read-incoming-hook
   (if (eq system-type 'windows-nt)
       '(nnheader-ms-strip-cr)
     nil)
@@ -243,13 +246,13 @@ running (\"xwatch\", etc.)
 
 Eg.
 
-\(add-hook 'nnmail-read-incoming-hook 
+\(add-hook 'nnmail-read-incoming-hook
           (lambda ()
-            (start-process \"mailsend\" nil 
+            (start-process \"mailsend\" nil
                            \"/local/bin/mailsend\" \"read\" \"mbox\")))
 
 If you have xwatch running, this will alert it that mail has been
-read.  
+read.
 
 If you use `display-time', you could use something like this:
 
@@ -330,14 +333,14 @@ To enable this, set `nnmail-split-methods' to `nnmail-split-fancy'.
 The format is this variable is SPLIT, where SPLIT can be one of
 the following:
 
-GROUP: Mail will be stored in GROUP (a string).  
+GROUP: Mail will be stored in GROUP (a string).
 
 \(FIELD VALUE SPLIT): If the message field FIELD (a regexp) contains
   VALUE (a regexp), store the messages as specified by SPLIT.
 
 \(| SPLIT...): Process each SPLIT expression until one of them matches.
   A SPLIT expression is said to match if it will cause the mail
-  message to be stored in one or more groups.  
+  message to be stored in one or more groups.
 
 \(& SPLIT...): Process each SPLIT expression.
 
@@ -347,7 +350,7 @@ GROUP: Mail will be stored in GROUP (a string).
 
 FIELD must match a complete field name.  VALUE must match a complete
 word according to the `nnmail-split-fancy-syntax-table' syntax table.
-You can use .* in the regexps to match partial field names or words.
+You can use \".*\" in the regexps to match partial field names or words.
 
 FIELD and VALUE can also be lisp symbols, in that case they are expanded
 as specified in `nnmail-split-abbrev-alist'.
@@ -471,7 +474,7 @@ parameter.  It should return nil, `warn' or `delete'."
   (concat
    (let ((dir (file-name-as-directory (expand-file-name dir))))
      ;; If this directory exists, we use it directly.
-     (if (or nnmail-use-long-file-names 
+     (if (or nnmail-use-long-file-names
             (file-directory-p (concat dir group)))
         (concat dir group "/")
        ;; If not, we translate dots into slashes.
@@ -532,7 +535,7 @@ parameter.  It should return nil, `warn' or `delete'."
       (delete-file nnmail-crash-box))
     (let ((inbox (file-truename (expand-file-name inbox)))
          (tofile (file-truename (expand-file-name nnmail-crash-box)))
-         movemail popmail errors)
+         movemail popmail errors result)
       (if (setq popmail (string-match
                         "^po:" (file-name-nondirectory inbox)))
          (setq inbox (file-name-nondirectory inbox))
@@ -563,7 +566,7 @@ parameter.  It should return nil, `warn' or `delete'."
            (message "Getting mail from %s..." inbox)))
        ;; Set TOFILE if have not already done so, and
        ;; rename or copy the file INBOX to TOFILE if and as appropriate.
-       (cond 
+       (cond
         ((file-exists-p tofile)
          ;; The crash box exists already.
          t)
@@ -581,17 +584,27 @@ parameter.  It should return nil, `warn' or `delete'."
                (buffer-disable-undo errors)
                (let ((default-directory "/"))
                  (if (nnheader-functionp nnmail-movemail-program)
-                     (funcall nnmail-movemail-program inbox tofile)
-                   (apply 
-                    'call-process
-                    (append
-                     (list
-                      (expand-file-name 
-                       nnmail-movemail-program exec-directory)
-                      nil errors nil inbox tofile)
-                     (when nnmail-internal-password
-                       (list nnmail-internal-password))))))
-               (if (not (buffer-modified-p errors))
+                     (condition-case err
+                         (progn
+                           (funcall nnmail-movemail-program inbox tofile)
+                           (setq result 0))
+                       (error
+                        (save-excursion
+                          (set-buffer errors)
+                          (insert (prin1-to-string err))
+                          (setq result 255))))
+                   (setq result
+                         (apply
+                          'call-process
+                          (append
+                           (list
+                            (expand-file-name
+                             nnmail-movemail-program exec-directory)
+                            nil errors nil inbox tofile)
+                           (when nnmail-internal-password
+                             (list nnmail-internal-password)))))))
+               (if (and (not (buffer-modified-p errors))
+                        (zerop result))
                    ;; No output => movemail won
                    (progn
                      (unless popmail
@@ -617,8 +630,8 @@ parameter.  It should return nil, `warn' or `delete'."
                    (when (looking-at "movemail: ")
                      (delete-region (point-min) (match-end 0)))
                    (unless (yes-or-no-p
-                            (format "movemail: %s.  Continue? "
-                                    (buffer-string)))
+                            (format "movemail: %s (%d return).  Continue? "
+                                    (buffer-string) result))
                      (error "%s" (buffer-string)))
                    (setq tofile nil)))))))
        (message "Getting mail from %s...done" inbox)
@@ -635,7 +648,7 @@ nn*-request-list should have been called before calling this function."
     (save-excursion
       (set-buffer nntp-server-buffer)
       (goto-char (point-min))
-      (while (re-search-forward 
+      (while (re-search-forward
              "^\\([^ \t]+\\)[ \t]+\\([0-9]+\\)[ \t]+\\([0-9]+\\)" nil t)
        ;; We create an alist with `(GROUP (LOW . HIGH))' elements.
        (push (list (match-string 1)
@@ -674,7 +687,7 @@ is a spool.  If not using procmail, return GROUP."
          (let ((procmail-group (substring (expand-file-name file)
                                           (match-beginning 1)
                                           (match-end 1))))
-           (if group 
+           (if group
                (if (string-equal group procmail-group)
                    group
                  nil)
@@ -707,12 +720,13 @@ is a spool.  If not using procmail, return GROUP."
       (goto-char (point-max))
       ;; Find the Message-ID header.
       (save-excursion
-       (if (re-search-backward "^Message-ID:[ \t]*\\(<[^>]*>\\)" nil t)
+       (if (re-search-backward
+            "^Message-ID[ \t]*:[ \n\t]*\\(<[^>]*>\\)" nil t)
            (setq message-id (buffer-substring (match-beginning 1)
                                               (match-end 1)))
          ;; There is no Message-ID here, so we create one.
          (save-excursion
-           (when (re-search-backward "^Message-ID:" nil t)
+           (when (re-search-backward "^Message-ID[ \t]*:" nil t)
              (beginning-of-line)
              (insert "Original-")))
          (forward-line -1)
@@ -720,10 +734,10 @@ is a spool.  If not using procmail, return GROUP."
                  "\n")))
       ;; Look for a Content-Length header.
       (if (not (save-excursion
-                (and (re-search-backward 
+                (and (re-search-backward
                       "^Content-Length:[ \t]*\\([0-9]+\\)" start t)
                      (setq content-length (string-to-int
-                                           (buffer-substring 
+                                           (buffer-substring
                                             (match-beginning 1)
                                             (match-end 1))))
                      ;; We destroy the header, since none of
@@ -743,7 +757,7 @@ is a spool.  If not using procmail, return GROUP."
          (setq do-search t)))
       (widen)
       ;; Go to the beginning of the next article - or to the end
-      ;; of the buffer.  
+      ;; of the buffer.
       (when do-search
        (if (re-search-forward "^\1f" nil t)
            (goto-char (match-beginning 0))
@@ -781,7 +795,7 @@ is a spool.  If not using procmail, return GROUP."
                       (forward-line 1)
                       (while (looking-at ">From ")
                         (forward-line 1))
-                      (looking-at "[^ \t:]+[ \t]*:")))
+                      (looking-at "[^ \n\t:]+[ \n\t]*:")))
            (setq found 'yes)))))
     (beginning-of-line)
     (eq found 'yes)))
@@ -810,7 +824,7 @@ is a spool.  If not using procmail, return GROUP."
                       (forward-line 1)
                       (while (looking-at ">From ")
                         (forward-line 1))
-                      (looking-at "[^ \t:]+[ \t]*:")))
+                      (looking-at "[^ \n\t:]+[ \n\t]*:")))
            (setq found 'yes)))))
     (beginning-of-line)
     (eq found 'yes)))
@@ -829,7 +843,7 @@ is a spool.  If not using procmail, return GROUP."
              end nil)
        ;; Find the end of the head.
        (narrow-to-region
-        start 
+        start
         (if (search-forward "\n\n" nil t)
             (1- (point))
           ;; This will never happen, but just to be on the safe side --
@@ -840,10 +854,10 @@ is a spool.  If not using procmail, return GROUP."
             (point))))
        ;; Find the Message-ID header.
        (goto-char (point-min))
-       (if (re-search-forward "^Message-ID:[ \t]*\\(<[^>]+>\\)" nil t)
+       (if (re-search-forward "^Message-ID[ \t]*:[ \n\t]*\\(<[^>]+>\\)" nil t)
            (setq message-id (match-string 1))
          (save-excursion
-           (when (re-search-forward "^Message-ID:" nil t)
+           (when (re-search-forward "^Message-ID[ \t]*:" nil t)
              (beginning-of-line)
              (insert "Original-")))
          ;; There is no Message-ID here, so we create one.
@@ -855,7 +869,7 @@ is a spool.  If not using procmail, return GROUP."
                  "^Content-Length:[ \t]*\\([0-9]+\\)" nil t))
            (setq content-length nil)
          (setq content-length (string-to-int (match-string 1)))
-         ;; We destroy the header, since none of the backends ever 
+         ;; We destroy the header, since none of the backends ever
          ;; use it, and we do not want to confuse other mailers by
          ;; having a (possibly) faulty header.
          (beginning-of-line)
@@ -885,7 +899,7 @@ is a spool.  If not using procmail, return GROUP."
                (t (setq end nil))))
        (if end
            (goto-char end)
-         ;; No Content-Length, so we find the beginning of the next 
+         ;; No Content-Length, so we find the beginning of the next
          ;; article or the end of the buffer.
          (goto-char head-end)
          (or (nnmail-search-unix-mail-delim)
@@ -913,7 +927,7 @@ is a spool.  If not using procmail, return GROUP."
        (setq start (point))
        ;; Find the end of the head.
        (narrow-to-region
-        start 
+        start
         (if (search-forward "\n\n" nil t)
             (1- (point))
           ;; This will never happen, but just to be on the safe side --
@@ -924,11 +938,11 @@ is a spool.  If not using procmail, return GROUP."
             (point))))
        ;; Find the Message-ID header.
        (goto-char (point-min))
-       (if (re-search-forward "^Message-ID:[ \t]*\\(<[^>]+>\\)" nil t)
+       (if (re-search-forward "^Message-ID[ \t]*:[ \n\t]*\\(<[^>]+>\\)" nil t)
            (setq message-id (match-string 1))
          ;; There is no Message-ID here, so we create one.
          (save-excursion
-           (when (re-search-backward "^Message-ID:" nil t)
+           (when (re-search-backward "^Message-ID[ \t]*:" nil t)
              (beginning-of-line)
              (insert "Original-")))
          (forward-line 1)
@@ -985,7 +999,7 @@ FUNC will be called with the buffer narrowed to each mail."
        (funcall exit-func))
       (kill-buffer (current-buffer)))))
 
-;; Mail crossposts suggested by Brian Edmonds <edmonds@cs.ubc.ca>. 
+;; Mail crossposts suggested by Brian Edmonds <edmonds@cs.ubc.ca>.
 (defun nnmail-article-group (func)
   "Look at the headers and return an alist of groups that match.
 FUNC will be called with the group name to determine the article number."
@@ -1020,12 +1034,12 @@ FUNC will be called with the group name to determine the article number."
                       (or (funcall nnmail-split-methods)
                           '("bogus"))
                     (error
-                     (message 
+                     (message
                       "Error in `nnmail-split-methods'; using `bogus' mail group")
                      (sit-for 1)
                      '("bogus")))))
              (unless (equal split '(junk))
-               ;; `nnmail-split-methods' is a function, so we just call 
+               ;; `nnmail-split-methods' is a function, so we just call
                ;; this function here and use the result.
                (setq group-art
                      (mapcar
@@ -1043,15 +1057,15 @@ FUNC will be called with the group name to determine the article number."
                             (re-search-backward (cadr method) nil t)
                           ;; Function to say whether this is a match.
                           (funcall (nth 1 method) (car method))))
-                      ;; Don't enter the article into the same 
+                      ;; Don't enter the article into the same
                       ;; group twice.
                       (not (assoc (car method) group-art)))
                  (push (cons (car method) (funcall func (car method)))
                        group-art))
-             ;; This is the final group, which is used as a 
+             ;; This is the final group, which is used as a
              ;; catch-all.
              (unless group-art
-               (setq group-art 
+               (setq group-art
                      (list (cons (car method)
                                  (funcall func (car method)))))))))
        ;; See whether the split methods returned `junk'.
@@ -1256,14 +1270,14 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
   (if (null nnmail-spool-file)
       ;; No spool file whatsoever.
       nil
-    (let* ((procmails 
+    (let* ((procmails
            ;; If procmail is used to get incoming mail, the files
            ;; are stored in this directory.
            (and (file-exists-p nnmail-procmail-directory)
                 (or (eq nnmail-spool-file 'procmail)
                     nnmail-use-procmail)
-                (directory-files 
-                 nnmail-procmail-directory 
+                (directory-files
+                 nnmail-procmail-directory
                  t (concat (if group (concat "^" group) "")
                            nnmail-procmail-suffix "$"))))
           (p procmails)
@@ -1273,13 +1287,13 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
                                0))
                    (list nnmail-crash-box))))
       ;; Remove any directories that inadvertently match the procmail
-      ;; suffix, which might happen if the suffix is "". 
+      ;; suffix, which might happen if the suffix is "".
       (while p
        (when (file-directory-p (car p))
          (setq procmails (delete (car p) procmails)))
        (setq p (cdr p)))
       ;; Return the list of spools.
-      (append 
+      (append
        crash
        (cond ((and group
                   (or (eq nnmail-spool-file 'procmail)
@@ -1291,9 +1305,9 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
              nil)
             ((listp nnmail-spool-file)
              (nconc
-              (apply 
+              (apply
                'nconc
-               (mapcar 
+               (mapcar
                 (lambda (file)
                   (if (and (not (string-match "^po:" file))
                            (file-directory-p file))
@@ -1304,7 +1318,7 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
             ((stringp nnmail-spool-file)
              (if (and (not (string-match "^po:" nnmail-spool-file))
                       (file-directory-p nnmail-spool-file))
-                 (nconc 
+                 (nconc
                   (nnheader-directory-regular-files nnmail-spool-file)
                   procmails)
                (cons nnmail-spool-file procmails)))
@@ -1313,22 +1327,22 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
             (t
              procmails))))))
 
-;; Activate a backend only if it isn't already activated. 
-;; If FORCE, re-read the active file even if the backend is 
+;; Activate a backend only if it isn't already activated.
+;; If FORCE, re-read the active file even if the backend is
 ;; already activated.
 (defun nnmail-activate (backend &optional force)
   (let (file timestamp file-time)
     (if (or (not (symbol-value (intern (format "%s-group-alist" backend))))
            force
            (and (setq file (ignore-errors
-                             (symbol-value (intern (format "%s-active-file" 
+                             (symbol-value (intern (format "%s-active-file"
                                                            backend)))))
                 (setq file-time (nth 5 (file-attributes file)))
                 (or (not
                      (setq timestamp
                            (condition-case ()
                                (symbol-value (intern
-                                              (format "%s-active-timestamp" 
+                                              (format "%s-active-timestamp"
                                                       backend)))
                              (error 'none))))
                     (not (consp timestamp))
@@ -1338,20 +1352,9 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
                          (> (nth 1 file-time) (nth 1 timestamp))))))
        (save-excursion
          (or (eq timestamp 'none)
-             (set (intern (format "%s-active-timestamp" backend)) 
-;;; dmoore@ucsd.edu 25.10.96
-;;; it's not always the case that current-time
-;;; does correspond to changes in the file's time.  So just compare
-;;; the file's new time against its own previous time.
-;;;               (current-time)
-                  file-time
-                  ))
-         (funcall (intern (format "%s-request-list" backend)))
-;;; dmoore@ucsd.edu 25.10.96
-;;; BACKEND-request-list already does this itself!
-;;;      (set (intern (format "%s-group-alist" backend)) 
-;;;           (nnmail-get-active))
-         ))
+             (set (intern (format "%s-active-timestamp" backend))
+                  file-time))
+         (funcall (intern (format "%s-request-list" backend)))))
     t))
 
 (defun nnmail-message-id ()
@@ -1369,8 +1372,8 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
               (buffer-name nnmail-cache-buffer)))
       ()                               ; The buffer is open.
     (save-excursion
-      (set-buffer 
-       (setq nnmail-cache-buffer 
+      (set-buffer
+       (setq nnmail-cache-buffer
             (get-buffer-create " *nnmail message-id cache*")))
       (buffer-disable-undo (current-buffer))
       (when (file-exists-p nnmail-message-id-cache-file)
@@ -1399,11 +1402,12 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
                           nnmail-message-id-cache-file nil 'silent)
       (set-buffer-modified-p nil)
       (setq nnmail-cache-buffer nil)
-      ;;(kill-buffer (current-buffer))
-      )))
+      (kill-buffer (current-buffer)))))
 
 (defun nnmail-cache-insert (id)
   (when nnmail-treat-duplicates
+    (unless (gnus-buffer-live-p nnmail-cache-buffer)
+      (nnmail-cache-open))
     (save-excursion
       (set-buffer nnmail-cache-buffer)
       (goto-char (point-max))
@@ -1416,6 +1420,12 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
       (goto-char (point-max))
       (search-backward id nil t))))
 
+(defun nnmail-fetch-field (header)
+  (save-excursion
+    (save-restriction
+      (message-narrow-to-head)
+      (message-fetch-field header))))
+
 (defun nnmail-check-duplication (message-id func artnum-func)
   (run-hooks 'nnmail-prepare-incoming-message-hook)
   ;; If this is a duplicate message, then we do not save it.
@@ -1440,17 +1450,12 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
       (setq group-art nil))
      ((eq action 'warn)
       ;; We insert a warning.
-      (let ((case-fold-search t)
-           (newid (nnmail-message-id)))
+      (let ((case-fold-search t))
        (goto-char (point-min))
-       (when (re-search-forward "^message-id:" nil t)
-         (beginning-of-line)
-         (insert "Original-"))
+       (re-search-forward "^message-id[ \t]*:" nil t)
        (beginning-of-line)
-       (insert 
-        "Message-ID: " newid "\n"
+       (insert
         "Gnus-Warning: This is a duplicate of message " message-id "\n")
-       (nnmail-cache-insert newid)
        (funcall func (setq group-art
                            (nreverse (nnmail-article-group artnum-func))))))
      (t
@@ -1502,24 +1507,24 @@ See the documentation for the variable `nnmail-split-fancy' for documentation."
            ;; is supposed to go to some specific group.
            (setq group (nnmail-get-split-group spool group-in))
            ;; We split the mail
-           (nnmail-split-incoming 
+           (nnmail-split-incoming
             nnmail-crash-box (intern (format "%s-save-mail" method))
             spool-func group (intern (format "%s-active-number" method)))
-           ;; Check whether the inbox is to be moved to the special tmp dir. 
+           ;; Check whether the inbox is to be moved to the special tmp dir.
            (setq incoming
-                 (nnmail-make-complex-temp-name 
-                  (expand-file-name 
+                 (nnmail-make-complex-temp-name
+                  (expand-file-name
                    (if nnmail-tmp-directory
-                       (concat 
+                       (concat
                         (file-name-as-directory nnmail-tmp-directory)
                         (file-name-nondirectory
                          (concat (file-name-as-directory temp) "Incoming")))
                      (concat (file-name-as-directory temp) "Incoming")))))
            (rename-file nnmail-crash-box incoming t)
            (push incoming incomings))))
-      ;; If we did indeed read any incoming spools, we save all info. 
+      ;; If we did indeed read any incoming spools, we save all info.
       (when incomings
-       (nnmail-save-active 
+       (nnmail-save-active
         (nnmail-get-value "%s-group-alist" method)
         (nnmail-get-value "%s-active-file" method))
        (when exit-func
@@ -1572,7 +1577,8 @@ If ARGS, PROMPT is used as an argument to `format'."
     (unless nnmail-read-passwd
       (if (load "passwd" t)
          (setq nnmail-read-passwd 'read-passwd)
-       (autoload 'ange-ftp-read-passwd "ange-ftp")
+       (unless (fboundp 'ange-ftp-read-passwd)
+         (autoload 'ange-ftp-read-passwd "ange-ftp"))
        (setq nnmail-read-passwd 'ange-ftp-read-passwd)))
     (funcall nnmail-read-passwd prompt)))
 
@@ -1581,7 +1587,7 @@ If ARGS, PROMPT is used as an argument to `format'."
   (save-restriction
     (message-narrow-to-head)
     (let ((case-fold-search t))
-      (unless (re-search-forward "^Message-ID:" nil t)
+      (unless (re-search-forward "^Message-ID[ \t]*:" nil t)
        (insert "Message-ID: " (nnmail-message-id) "\n")))))
 
 (defun nnmail-write-region (start end filename &optional append visit lockname)
@@ -1673,15 +1679,17 @@ If ARGS, PROMPT is used as an argument to `format'."
              his nil)))
     found))
 
+(eval-and-compile
+  (autoload 'pop3-movemail "pop3"))
+
 (defun nnmail-pop3-movemail (inbox crashbox)
   "Function to move mail from INBOX on a pop3 server to file CRASHBOX."
-  (require 'pop3)
   (let ((pop3-maildrop
          (substring inbox (match-end (string-match "^po:" inbox)))))
     (pop3-movemail crashbox)))
 
 (run-hooks 'nnmail-load-hook)
-           
+
 (provide 'nnmail)
 
 ;;; nnmail.el ends here