]> git.eshelyaron.com Git - emacs.git/commitdiff
Improvements to vc-bzr conffile handling and pull/merge support.
authorChong Yidong <cyd@stupidchicken.com>
Sat, 19 Feb 2011 21:23:51 +0000 (16:23 -0500)
committerChong Yidong <cyd@stupidchicken.com>
Sat, 19 Feb 2011 21:23:51 +0000 (16:23 -0500)
* vc/vc-bzr.el (vc-bzr--branch-conf): Function deleted.
(vc-bzr-branch-conf): New function, similar to vc-bzr--branch-conf
but returning an alist.  Ignore comments in bzr conffile.
(vc-bzr-pull, vc-bzr-merge-branch): Use vc-bzr-branch-conf.
(vc-bzr-error-regex-alist): New var.
(vc-bzr-merge-branch): Use it to highlight the pull/merge buffer.

* progmodes/compile.el (compilation--flush-directory-cache):
Handle the case where cdr of compilation--flush-directory-cache
points to no buffer, which can occur if we previously switched to
compilation-mode in a pregenerated buffer.

* vc/vc-dispatcher.el (vc-do-async-command): Bind
inhibit-read-only to t.

lisp/ChangeLog
lisp/progmodes/compile.el
lisp/vc/vc-bzr.el
lisp/vc/vc-dispatcher.el

index 26823aa6bcb893cf0be30d0a301297b69b98935f..e0255ceb90c77d5f58a589111af8680a4ed12994 100644 (file)
@@ -1,3 +1,20 @@
+2011-02-19  Chong Yidong  <cyd@stupidchicken.com>
+
+       * vc/vc-bzr.el (vc-bzr--branch-conf): Function deleted.
+       (vc-bzr-branch-conf): New function, similar to vc-bzr--branch-conf
+       but returning an alist.  Ignore comments in bzr conffile.
+       (vc-bzr-pull, vc-bzr-merge-branch): Use vc-bzr-branch-conf.
+       (vc-bzr-error-regex-alist): New var.
+       (vc-bzr-merge-branch): Use it to highlight the pull/merge buffer.
+
+       * vc/vc-dispatcher.el (vc-do-async-command): Bind
+       inhibit-read-only to t.
+
+       * progmodes/compile.el (compilation--flush-directory-cache):
+       Handle the case where cdr of compilation--flush-directory-cache
+       points to no buffer, which can occur if we previously switched to
+       compilation-mode in a pregenerated buffer.
+
 2011-02-19  Kenichi Handa  <handa@m17n.org>
 
        * mail/rmailmm.el (rmail-mime-find-header-encoding): Be sure to
index b41eb82e27d4fa30c1754919948776ec69d6e2bc..88f418f934a42a3ee9d06b28c8ab8958aa49111b 100644 (file)
@@ -838,6 +838,7 @@ POS and RES.")
    ((or (not compilation--previous-directory-cache)
         (<= (car compilation--previous-directory-cache) start)))
    ((or (not (cdr compilation--previous-directory-cache))
+       (null (marker-buffer (cdr compilation--previous-directory-cache)))
         (<= (cdr compilation--previous-directory-cache) start))
     (set-marker (car compilation--previous-directory-cache) start))
    (t (setq compilation--previous-directory-cache nil))))
index a36fdc60d154d09d5c57ef3fb7e10c613b83eb72..5e6e054924c622fb4723dac2a71298682f78bfc4 100644 (file)
@@ -141,12 +141,20 @@ Use the current Bzr root directory as the ROOT argument to
       (let ((root (vc-find-root file vc-bzr-admin-checkout-format-file)))
        (when root (vc-file-setprop file 'bzr-root root)))))
 
-(defun vc-bzr--branch-conf (file)
-  "Return the Bzr branch config for file FILE, as a string."
-  (with-temp-buffer
-    (insert-file-contents
-     (expand-file-name vc-bzr-admin-branchconf (vc-bzr-root file)))
-    (buffer-string)))
+(defun vc-bzr-branch-conf (file)
+  "Return the Bazaar branch settings for file FILE, as an alist.
+Each element of the returned alist has the form (NAME . VALUE),
+which are the name and value of a Bazaar setting, as strings.
+
+The settings are read from the file \".bzr/branch/branch.conf\"
+in the repository root directory of FILE."
+  (let (settings)
+    (with-temp-buffer
+      (insert-file-contents
+       (expand-file-name vc-bzr-admin-branchconf (vc-bzr-root file)))
+      (while (re-search-forward "^\\([^#=][^=]*?\\) *= *\\(.*\\)$" nil t)
+       (push (cons (match-string 1) (match-string 2)) settings)))
+    settings))
 
 (require 'sha1)                         ;For sha1-program
 
@@ -276,6 +284,13 @@ Use the current Bzr root directory as the ROOT argument to
     (when rootdir
          (file-relative-name filename* rootdir))))
 
+(defvar vc-bzr-error-regex-alist
+  '(("^\\( M[* ]\\|+N \\|-D \\|\\|  \\*\\|R[M ] \\) \\(.+\\)" 2 nil nil 1)
+    ("^C  \\(.+\\)" 2)
+    ("^Text conflict in \\(.+\\)" 1 nil nil 2)
+    ("^Using saved parent location: \\(.+\\)" 1 nil nil 0))
+  "Value of `compilation-error-regexp-alist' in *vc-bzr* buffers.")
+
 (defun vc-bzr-pull (prompt)
   "Pull changes into the current Bzr branch.
 Normally, this runs \"bzr pull\".  However, if the branch is a
@@ -283,19 +298,18 @@ bound branch, run \"bzr update\" instead.  If there is no default
 location from which to pull or update, or if PROMPT is non-nil,
 prompt for the Bzr command to run."
   (let* ((vc-bzr-program vc-bzr-program)
-        (branch-conf (vc-bzr--branch-conf default-directory))
+        (branch-conf (vc-bzr-branch-conf default-directory))
         ;; Check whether the branch is bound.
-        (bound (string-match "^bound\\s-*=\\s-*True" branch-conf))
+        (bound (assoc "bound" branch-conf))
+        (bound (and bound (equal "true" (downcase (cdr bound)))))
         ;; If we need to do a "bzr pull", check for a parent.  If it
         ;; does not exist, bzr will need a pull location.
-        (parent (unless bound
-                  (string-match
-                   "^parent_location\\s-*=\\s-*[^\n[:space:]]+"
-                   branch-conf)))
+        (has-parent (unless bound
+                      (assoc "parent_location" branch-conf)))
         (command (if bound "update" "pull"))
         args)
     ;; If necessary, prompt for the exact command.
-    (when (or prompt (not (or bound parent)))
+    (when (or prompt (not (or bound has-parent)))
       (setq args (split-string
                  (read-shell-command
                   "Bzr pull command: "
@@ -305,28 +319,33 @@ prompt for the Bzr command to run."
       (setq vc-bzr-program (car  args)
            command        (cadr args)
            args           (cddr args)))
-    (vc-set-async-update
-     (apply 'vc-bzr-async-command command args))))
+    (let ((buf (apply 'vc-bzr-async-command command args)))
+      (with-current-buffer buf
+       (vc-exec-after
+        `(progn
+           (let ((compilation-error-regexp-alist
+                  vc-bzr-error-regex-alist))
+             (compilation-mode))
+           (set (make-local-variable 'compilation-error-regexp-alist)
+                vc-bzr-error-regex-alist))))
+      (vc-set-async-update buf))))
 
 (defun vc-bzr-merge-branch ()
   "Merge another Bzr branch into the current one.
 Prompt for the Bzr command to run, providing a pre-defined merge
 source (an upstream branch or a previous merge source) as a
 default if it is available."
-  (let* ((branch-conf (vc-bzr--branch-conf default-directory))
+  (let* ((branch-conf (vc-bzr-branch-conf default-directory))
         ;; "bzr merge" without an argument defaults to submit_branch,
         ;; then parent_location.  Extract the specific location and
         ;; add it explicitly to the command line.
+        (setting nil)
         (location
          (cond
-          ((string-match
-            "^submit_branch\\s-*=\\s-*\\(?:file://\\)?\\([^\n[:space:]]+\\)$"
-            branch-conf)
-           (match-string 1 branch-conf))
-          ((string-match
-            "^parent_location\\s-*=\\s-*\\(?:file://\\)?\\([^\n[:space:]]+\\)$"
-            branch-conf)
-           (match-string 1 branch-conf))))
+          ((setq setting (assoc "submit_branch" branch-conf))
+           (cdr setting))
+          ((setq setting (assoc "parent_location" branch-conf))
+           (cdr setting))))
         (cmd
          (split-string
           (read-shell-command
@@ -338,8 +357,16 @@ default if it is available."
         (vc-bzr-program (car  cmd))
         (command        (cadr cmd))
         (args           (cddr cmd)))
-    (vc-set-async-update
-     (apply 'vc-bzr-async-command command args))))
+    (let ((buf (apply 'vc-bzr-async-command command args)))
+      (with-current-buffer buf
+       (vc-exec-after
+        `(progn
+           (let ((compilation-error-regexp-alist
+                  vc-bzr-error-regex-alist))
+             (compilation-mode))
+           (set (make-local-variable 'compilation-error-regexp-alist)
+                vc-bzr-error-regex-alist))))
+      (vc-set-async-update buf))))
 
 (defun vc-bzr-status (file)
   "Return FILE status according to Bzr.
index c4e0dbfadac224be9a1889f8d68473ea1e9f3f05..388d4c94a08e34c8cb2ccf37fdda2fa63db79a36 100644 (file)
@@ -363,6 +363,7 @@ of a buffer, which is created.
 ROOT should be the directory in which the command should be run.
 Display the buffer in some window, but don't select it."
   (let* ((dir default-directory)
+        (inhibit-read-only t)
         window new-window-start)
     (setq buffer (get-buffer-create buffer))
     (if (get-buffer-process buffer)