unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Rohan Hart <rohan.hart@fronde.com>
To: Lars Ingebrigtsen <larsi@gnus.org>
Cc: 27453@debbugs.gnu.org
Subject: bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
Date: Wed, 24 Jul 2019 08:57:09 +1200	[thread overview]
Message-ID: <CAKQntwK+iBTybv31dOa=kuW30ZWs7GUUZyx_SwEnkQifOpEqYg@mail.gmail.com> (raw)
In-Reply-To: <87o91l0yxc.fsf@mouse.gnus.org>

[-- Attachment #1: Type: text/plain, Size: 2405 bytes --]

vc-darcs is the emacs package for darcs integration with vc.

If you wished to support very old versions of darcs then keep that complex
match and put the new one before it. Not that I've tested that config - I
just override the entire log-view-message-re as I don't use any of those
other version control systems

thanks

On Tue, 23 Jul 2019 at 23:32, Lars Ingebrigtsen <larsi@gnus.org> wrote:

> Rohan Hart <rohan.hart@fronde.com> writes:
>
> > Well sure, I only raised this as log-view states:
> >
> > ;; Currently supports the format output by:
> > ;; SCCS, RCS, CVS, Subversion, DaRCS, and Mercurial.
> >
> > And later mentions code related to both darcsum and vc-darcs.
>
> Then perhaps it does make sense to add the regexp to log-view-message-re
> after all.  The current definition is:
>
> (defvar log-view-message-re
>   (concat "^\\(?:revision \\(?1:[.0-9]+\\)\\(?:\t.*\\)?" ; RCS and CVS.
>           "\\|r\\(?1:[0-9]+\\) | .* | .*"                ; Subversion.
>           "\\|D \\(?1:[.0-9]+\\) .*"                     ; SCCS.
>           ;; Darcs doesn't have revision names.  VC-darcs uses patch names
>           ;; instead.  Darcs patch names are hashcodes, which do not appear
>           ;; in the log output :-(, but darcs accepts any prefix of the log
>           ;; message as a patch name, so we match the first line of the log
>           ;; message.
>           ;; First loosely match the date format.
>           (concat "\\|[^ \n].*[^0-9\n][0-9][0-9]:[0-9][0-9][^0-9\n].*[^
> \n]"
>                   ;;Email of user and finally Msg, used as revision name.
>                   "  .*@.*\n\\(?:  \\* \\(?1:.*\\)\\)?")
>           "\\)$")
>
> Is VC-darcs something else than darcs?  Or is that an Emacs package?
>
> --
> (domestic pets only, the antidote for overdose, milk.)
>    bloggy blog: http://lars.ingebrigtsen.no
>


-- 

*Rohan Hart*
Technical Lead Developer
Fronde


[image: Fronde Logo]

Level 1, 222 Lambton Quay | PO Box 11-545 | Wellington 6011, New Zealand
m. +64 21 937 533  | e. rohan.hart@fronde.com | w. www.fronde.com

-- 
The information contained in this email is privileged and confidential and 
intended for the addressee only. If you are not the intended recipient, 
please respect that confidentiality and do not disclose, copy or make use 
of its contents. If received in error, please destroy this email and 
contact the sender immediately. Thank you.

[-- Attachment #2: Type: text/html, Size: 4398 bytes --]

  reply	other threads:[~2019-07-23 20:57 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-23  0:49 bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format Rohan Hart
2019-07-21 17:40 ` Lars Ingebrigtsen
2019-07-21 20:36   ` Rohan Hart
2019-07-22 12:28     ` Lars Ingebrigtsen
2019-07-22 20:42       ` Rohan Hart
2019-07-23 11:31         ` Lars Ingebrigtsen
2019-07-23 20:57           ` Rohan Hart [this message]
2019-07-24 12:32             ` Lars Ingebrigtsen
2020-08-20 16:01               ` Lars Ingebrigtsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAKQntwK+iBTybv31dOa=kuW30ZWs7GUUZyx_SwEnkQifOpEqYg@mail.gmail.com' \
    --to=rohan.hart@fronde.com \
    --cc=27453@debbugs.gnu.org \
    --cc=larsi@gnus.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).