(mail-alias-modtime): New variable.
[bpt/emacs.git] / lisp / tar-mode.el
index 00fff38..bddd32a 100644 (file)
@@ -1,6 +1,6 @@
 ;;; tar-mode.el --- simple editing of tar files from GNU emacs
 
-;;; Copyright (C) 1990, 1991, 1993 Free Software Foundation, Inc.
+;;; Copyright (C) 1990, 1991, 1993, 1994 Free Software Foundation, Inc.
 
 ;; Author: Jamie Zawinski <jwz@lucid.com>
 ;; Created: 04 Apr 1990
 ;;; o  Block files, sparse files, continuation files, and the various header 
 ;;;    types aren't editable.  Actually I don't know that they work at all.
 
+;;; Rationale:
+
+;;; Why does tar-mode edit the file itself instead of using tar?
+
+;;; That means that you can edit tar files which you don't have room for
+;;; on your local disk.
+
+;;; I don't know about recent features in gnu tar, but old versions of tar
+;;; can't replace a file in the middle of a tar file with a new version.
+;;; Tar-mode can.  I don't think tar can do things like chmod the subfiles.
+;;; An implementation which involved unpacking and repacking the file into
+;;; some scratch directory would be very wasteful, and wouldn't be able to
+;;; preserve the file owners.
+
 ;;; Code:
 
 (defvar tar-anal-blocksize 20
@@ -202,7 +216,7 @@ the file never exists on disk.")
 (defconst tar-dmin-offset (+ tar-dmaj-offset 8))
 (defconst tar-end-offset (+ tar-dmin-offset 8))
 
-(defun tokenize-tar-header-block (string)
+(defun tar-header-block-tokenize (string)
   "Return a `tar-header' structure.
 This is a list of name, mode, uid, gid, size, 
 write-date, checksum, link-type, and link-name."
@@ -269,7 +283,7 @@ write-date, checksum, link-type, and link-name."
   (tar-parse-octal-integer string))
 
 
-(defun checksum-tar-header-block (string)
+(defun tar-header-block-checksum (string)
   "Compute and return a tar-acceptable checksum for this block."
   (let* ((chk-field-start tar-chk-offset)
         (chk-field-end (+ chk-field-start 8))
@@ -286,14 +300,14 @@ write-date, checksum, link-type, and link-name."
            i (1+ i)))
     (+ sum (* 32 8))))
 
-(defun check-tar-header-block-checksum (hblock desired-checksum file-name)
+(defun tar-header-block-check-checksum (hblock desired-checksum file-name)
   "Beep and print a warning if the checksum doesn't match."
-  (if (not (= desired-checksum (checksum-tar-header-block hblock)))
+  (if (not (= desired-checksum (tar-header-block-checksum hblock)))
       (progn (beep) (message "Invalid checksum for file %s!" file-name))))
 
-(defun recompute-tar-header-block-checksum (hblock)
+(defun tar-header-block-recompute-checksum (hblock)
   "Modifies the given string to have a valid checksum field."
-  (let* ((chk (checksum-tar-header-block hblock))
+  (let* ((chk (tar-header-block-checksum hblock))
         (chk-string (format "%6o" chk))
         (l (length chk-string)))
     (aset hblock 154 0)
@@ -317,7 +331,7 @@ write-date, checksum, link-type, and link-name."
   (if (zerop (logand 2048 mode)) nil (aset string (+ start 5) ?s))
   string)
 
-(defun summarize-tar-header-block (tar-hblock &optional mod-p)
+(defun tar-header-block-summarize (tar-hblock &optional mod-p)
   "Returns a line similar to the output of `tar -vtf'."
   (let ((name (tar-header-name tar-hblock))
        (mode (tar-header-mode tar-hblock))
@@ -387,7 +401,7 @@ is visible (and the real data of the buffer is hidden)."
        (tokens nil))
     (while (not (eq tokens 'empty-tar-block))
       (let* ((hblock (buffer-substring pos (+ pos 512))))
-       (setq tokens (tokenize-tar-header-block hblock))
+       (setq tokens (tar-header-block-tokenize hblock))
        (setq pos (+ pos 512))
        (message "parsing tar file...%s%%"
                 ;(/ (* pos 100) bs)   ; this gets round-off lossage
@@ -405,8 +419,8 @@ is visible (and the real data of the buffer is hidden)."
                       (tar-header-name tokens) size))
            ;
            ; This is just too slow.  Don't really need it anyway....
-           ;(check-tar-header-block-checksum
-           ;  hblock (checksum-tar-header-block hblock)
+           ;(tar-header-block-check-checksum
+           ;  hblock (tar-header-block-checksum hblock)
            ;  (tar-header-name tokens))
            
            (setq result (cons (make-tar-desc pos tokens) result))
@@ -425,7 +439,7 @@ is visible (and the real data of the buffer is hidden)."
     (let ((buffer-read-only nil))
       (tar-dolist (tar-desc tar-parse-info)
        (insert-string
-         (summarize-tar-header-block (tar-desc-tokens tar-desc)))
+         (tar-header-block-summarize (tar-desc-tokens tar-desc)))
        (insert-string "\n"))
       (make-local-variable 'tar-header-offset)
       (setq tar-header-offset (point))
@@ -669,9 +683,15 @@ save your changes to disk."
                (set-buffer buffer)
                (insert-buffer-substring tar-buffer start end)
                (goto-char 0)
-               (set-visited-file-name name) ; give it a name to decide mode.
+               ;; Give it a name for list-buffers and to decide mode.
+               ;; Set buffer-file-name by hand first
+               ;; so that set-visited-file-name won't lock the filename.
+               (setq buffer-file-name
+                     (expand-file-name (concat tarname ":" name)))
+               (set-visited-file-name buffer-file-name)
                (normal-mode)  ; pick a mode.
-               (set-visited-file-name nil)  ; nuke the name - not meaningful.
+;;; Without a file name, save-buffer doesn't work.
+;;;            (set-visited-file-name nil)  ; nuke the name - not meaningful.
                (rename-buffer bufname)
                
                (make-local-variable 'tar-superior-buffer)
@@ -679,11 +699,6 @@ save your changes to disk."
                (setq tar-superior-buffer tar-buffer)
                (setq tar-superior-descriptor descriptor)
 
-               ;; Since the "real" file name is not in buffer-file-name,
-               ;; put it here for list-buffers.
-               (make-local-variable 'list-buffers-directory)
-               (setq list-buffers-directory name)
-
                (tar-subfile-mode 1)
                
                (setq buffer-read-only read-only-p)
@@ -749,23 +764,10 @@ the current tar-entry."
         (size (tar-header-size tokens))
         (start (+ (tar-desc-data-start descriptor) tar-header-offset -1))
         (end (+ start size)))
-    (let* ((tar-buffer (current-buffer))
-          buffer)
-      (unwind-protect
-         (progn
-           (setq buffer (generate-new-buffer "*tar-copy-tmp*"))
-           (widen)
-           (save-excursion
-             (set-buffer buffer)
-             (insert-buffer-substring tar-buffer start end)
-             (set-buffer-modified-p nil) ; in case we abort
-             (write-file to-file)
-             (message "Copied tar entry %s to %s" name to-file)
-             (set-buffer tar-buffer)))
-       (narrow-to-region 1 tar-header-offset)
-       (if buffer (kill-buffer buffer)))
-      )))
-
+    (save-restriction
+      (widen)
+      (write-region start end to-file))
+    (message "Copied tar entry %s to %s" name to-file)))
 
 (defun tar-flag-deleted (p &optional unflag)
   "*In Tar mode, mark this sub-file to be deleted from the tar file.
@@ -965,7 +967,7 @@ for this to be permanent."
          (let ((p (point)))
            (forward-line 1)
            (delete-region p (point))
-           (insert (summarize-tar-header-block tokens) "\n")
+           (insert (tar-header-block-summarize tokens) "\n")
            (setq tar-header-offset (point-max)))
          
          (widen)
@@ -977,7 +979,7 @@ for this to be permanent."
            (insert new-data-string) ; <--
            ;;
            ;; compute a new checksum and insert it.
-           (let ((chk (checksum-tar-header-block
+           (let ((chk (tar-header-block-checksum
                        (buffer-substring start (+ start 512)))))
              (goto-char (+ start tar-chk-offset))
              (delete-region (point) (+ (point) 8))
@@ -987,7 +989,7 @@ for this to be permanent."
              (tar-setf (tar-header-checksum tokens) chk)
              ;;
              ;; ok, make sure we didn't botch it.
-             (check-tar-header-block-checksum
+             (tar-header-block-check-checksum
                (buffer-substring start (+ start 512))
                chk (tar-header-name tokens))
              )))
@@ -1065,7 +1067,7 @@ to make your changes permanent."
                (insert ? ))
              ;;
              ;; compute a new checksum and insert it.
-             (let ((chk (checksum-tar-header-block
+             (let ((chk (tar-header-block-checksum
                          (buffer-substring header-start data-start))))
                (goto-char (+ header-start tar-chk-offset))
                (delete-region (point) (+ (point) 8))
@@ -1081,10 +1083,14 @@ to make your changes permanent."
              (next-line position)
              (beginning-of-line)
              (let ((p (point))
+                   after
                    (m (set-marker (make-marker) tar-header-offset)))
                (forward-line 1)
-               (delete-region p (point))
-               (insert-before-markers (summarize-tar-header-block tokens t) "\n")
+               (setq after (point))
+               ;; Insert the new text after the old, before deleting,
+               ;; to preserve the window start.
+               (insert-before-markers (tar-header-block-summarize tokens t) "\n")
+               (delete-region p after)
                (setq tar-header-offset (marker-position m)))
              )))
        ;; after doing the insertion, add any final padding that may be necessary.
@@ -1093,7 +1099,7 @@ to make your changes permanent."
     (set-buffer-modified-p t)   ; mark the tar file as modified
     (set-buffer subfile)
     (set-buffer-modified-p nil) ; mark the tar subfile as unmodified
-    (message "saved into tar-buffer \"%s\" - remember to save that buffer!"
+    (message "saved into tar-buffer `%s' -- remember to save that buffer!"
             (buffer-name tar-superior-buffer))
     ;; Prevent ordinary saving from happening.
     t)))