unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
@ 2017-06-23  0:49 Rohan Hart
  2019-07-21 17:40 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Rohan Hart @ 2017-06-23  0:49 UTC (permalink / raw)
  To: 27453

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

Darcs now shows the internal hash for each patch in a format like:

patch f4128969c4700a0b34414390d04159cb6b11b4e5
Author: bob <bob>
Date:   Sun May 21 22:44:55 NZST 2017
  * patch name
  - further details

Adding the regex "\\|^patch \\(?1:[.0-9a-f]+\\)" to log-view-message-re
seems to be sufficient to use the hash.

cheers
Rohan
-- 
| Rohan Hart | Consultant | Fronde Systems Group Ltd
| 3 Queens Wharf | PO Box 11545 | Wellington 6142, New Zealand
| p. +64 4 499 3000 | ddi. +64 4 439 4080 | 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: 1325 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
  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
  0 siblings, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-21 17:40 UTC (permalink / raw)
  To: Rohan Hart; +Cc: 27453

Rohan Hart <rohan.hart@fronde.com> writes:

> Darcs now shows the internal hash for each patch in a format like:
>
> patch f4128969c4700a0b34414390d04159cb6b11b4e5
> Author: bob <bob>
> Date:   Sun May 21 22:44:55 NZST 2017
>   * patch name
>   - further details
>
> Adding the regex "\\|^patch \\(?1:[.0-9a-f]+\\)" to log-view-message-re
> seems to be sufficient to use the hash.

Hm...  Unless I'm mistaken, Emacs doesn't support Darcs?  I ran

$ darcs clone https://hub.darcs.net/onetom/0wnDB

and Emacs then claimed that the files in that directory weren't under
source control?  So I'm unsure how this would help...

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
  2019-07-21 17:40 ` Lars Ingebrigtsen
@ 2019-07-21 20:36   ` Rohan Hart
  2019-07-22 12:28     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Rohan Hart @ 2019-07-21 20:36 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 27453

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

Hi Lars

Emacs doesn't support Darcs out of the box. See
https://www.emacswiki.org/emacs/DaRcs

cheers
Rohan

On Mon, 22 Jul 2019 at 05:40, Lars Ingebrigtsen <larsi@gnus.org> wrote:

> Rohan Hart <rohan.hart@fronde.com> writes:
>
> > Darcs now shows the internal hash for each patch in a format like:
> >
> > patch f4128969c4700a0b34414390d04159cb6b11b4e5
> > Author: bob <bob>
> > Date:   Sun May 21 22:44:55 NZST 2017
> >   * patch name
> >   - further details
> >
> > Adding the regex "\\|^patch \\(?1:[.0-9a-f]+\\)" to log-view-message-re
> > seems to be sufficient to use the hash.
>
> Hm...  Unless I'm mistaken, Emacs doesn't support Darcs?  I ran
>
> $ darcs clone https://hub.darcs.net/onetom/0wnDB
>
> and Emacs then claimed that the files in that directory weren't under
> source control?  So I'm unsure how this would help...
>
> --
> (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: 3574 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
  2019-07-21 20:36   ` Rohan Hart
@ 2019-07-22 12:28     ` Lars Ingebrigtsen
  2019-07-22 20:42       ` Rohan Hart
  0 siblings, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-22 12:28 UTC (permalink / raw)
  To: Rohan Hart; +Cc: 27453

Rohan Hart <rohan.hart@fronde.com> writes:

> Emacs doesn't support Darcs out of the box. See
> https://www.emacswiki.org/emacs/DaRcs

Wouldn't it then be up to these external packages to extend
`log-view-message-re' to match the formats?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
  2019-07-22 12:28     ` Lars Ingebrigtsen
@ 2019-07-22 20:42       ` Rohan Hart
  2019-07-23 11:31         ` Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Rohan Hart @ 2019-07-22 20:42 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 27453

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

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.

I'll raise it with the other packages

cheers

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

> Rohan Hart <rohan.hart@fronde.com> writes:
>
> > Emacs doesn't support Darcs out of the box. See
> > https://www.emacswiki.org/emacs/DaRcs
>
> Wouldn't it then be up to these external packages to extend
> `log-view-message-re' to match the formats?
>
> --
> (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: 3159 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
  2019-07-22 20:42       ` Rohan Hart
@ 2019-07-23 11:31         ` Lars Ingebrigtsen
  2019-07-23 20:57           ` Rohan Hart
  0 siblings, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-23 11:31 UTC (permalink / raw)
  To: Rohan Hart; +Cc: 27453

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





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
  2019-07-23 11:31         ` Lars Ingebrigtsen
@ 2019-07-23 20:57           ` Rohan Hart
  2019-07-24 12:32             ` Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Rohan Hart @ 2019-07-23 20:57 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 27453

[-- 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 --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
  2019-07-23 20:57           ` Rohan Hart
@ 2019-07-24 12:32             ` Lars Ingebrigtsen
  2020-08-20 16:01               ` Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-24 12:32 UTC (permalink / raw)
  To: Rohan Hart; +Cc: 27453

Rohan Hart <rohan.hart@fronde.com> writes:

> 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

As this isn't something I can test myself, can you post a patch to
log-view.el that you've verified works for you?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#27453: 25.1; vc/log-view.el log-view-message-re doesn't support newer Darcs log format
  2019-07-24 12:32             ` Lars Ingebrigtsen
@ 2020-08-20 16:01               ` Lars Ingebrigtsen
  0 siblings, 0 replies; 9+ messages in thread
From: Lars Ingebrigtsen @ 2020-08-20 16:01 UTC (permalink / raw)
  To: Rohan Hart; +Cc: 27453

Lars Ingebrigtsen <larsi@gnus.org> writes:

> Rohan Hart <rohan.hart@fronde.com> writes:
>
>> 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
>
> As this isn't something I can test myself, can you post a patch to
> log-view.el that you've verified works for you?

More information was requested, but no response was given within a yes,
so I'm closing this bug report.  If this is something that should still
be worked on, please respond to this email and we'll reopen the bug
report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-08-20 16:01 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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
2019-07-24 12:32             ` Lars Ingebrigtsen
2020-08-20 16:01               ` Lars Ingebrigtsen

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).