diff options
author | Dmitry Gutov <dgutov@yandex.ru> | 2015-04-11 07:09:28 +0300 |
---|---|---|
committer | Dmitry Gutov <dgutov@yandex.ru> | 2015-04-11 07:09:56 +0300 |
commit | 878058df46b593dd2afaad2086d8ff10bff367c2 (patch) | |
tree | 4ab8a61eac7833a14c08595247d58d5c73e67065 /lisp/vc/log-edit.el | |
parent | e3ee455d2992f4adddcd27196b47273f6cacb4e1 (diff) | |
download | emacs-878058df46b593dd2afaad2086d8ff10bff367c2.tar.gz |
Extract ChangeLog entries when committing a directory
* lisp/vc/vc-dispatcher.el (vc-log-edit): Update FIXME comment.
* lisp/vc/log-edit.el (log-edit-changelog-insert-entries):
Add a FIXME comment.
(log-edit-changelog-entries): Extract from
`log-edit-changelog-entries', handle FILE being a directory
(http://lists.gnu.org/archive/html/emacs-devel/2015-04/msg00555.html).
Diffstat (limited to 'lisp/vc/log-edit.el')
-rw-r--r-- | lisp/vc/log-edit.el | 38 |
1 files changed, 23 insertions, 15 deletions
diff --git a/lisp/vc/log-edit.el b/lisp/vc/log-edit.el index e74ca7267c5..f82c7e9052e 100644 --- a/lisp/vc/log-edit.el +++ b/lisp/vc/log-edit.el @@ -917,21 +917,8 @@ where LOGBUFFER is the name of the ChangeLog buffer, and each (log-edit-narrow-changelog) (goto-char (point-min)) - ;; Search for the name of FILE relative to the ChangeLog. If that - ;; doesn't occur anywhere, they're not using full relative - ;; filenames in the ChangeLog, so just look for FILE; we'll accept - ;; some false positives. - (let ((pattern (file-relative-name - file (file-name-directory changelog-file-name)))) - (if (or (string= pattern "") - (not (save-excursion - (search-forward pattern nil t)))) - (setq pattern (file-name-nondirectory file))) - - (setq pattern (concat "\\(^\\|[^[:alnum:]]\\)" - (regexp-quote pattern) - "\\($\\|[^[:alnum:]]\\)")) - + (let ((pattern (log-edit-changelog--pattern file + changelog-file-name))) (let (texts (pos (point))) (while (and (not (eobp)) (re-search-forward pattern nil t)) @@ -946,6 +933,25 @@ where LOGBUFFER is the name of the ChangeLog buffer, and each (cons (current-buffer) texts))))))))) +(defun log-edit-changelog--pattern (file changelog-file-name) + (if (eq (aref file (1- (length file))) ?/) + ;; Match any files inside this directory. + (concat "^\t\\* " (unless (string= file "./") file)) + ;; Search for the name of FILE relative to the ChangeLog. If that + ;; doesn't occur anywhere, they're not using full relative + ;; filenames in the ChangeLog, so just look for FILE; we'll accept + ;; some false positives. + (let ((pattern (file-relative-name + file (file-name-directory changelog-file-name)))) + ;; FIXME: When can the above return an empty string? + (if (or (string= pattern "") + (not (save-excursion + (search-forward pattern nil t)))) + (setq pattern (file-name-nondirectory file))) + (setq pattern (concat "\\(^\\|[^[:alnum:]]\\)" + (regexp-quote pattern) + "\\($\\|[^[:alnum:]]\\)"))))) + (defun log-edit-changelog-insert-entries (buffer beg end &rest files) "Insert the text from BUFFER between BEG and END. Rename relative filenames in the ChangeLog entry as FILES." @@ -957,6 +963,8 @@ Rename relative filenames in the ChangeLog entry as FILES." (setq bound (point-marker)) (when log-name (dolist (f files) + ;; FIXME: f can be a directory, a (possibly indirect) parent + ;; of the ChangeLog file. (save-excursion (goto-char opoint) (when (re-search-forward |