GNU bug report logs -
#6968
23.1; vc-hg-next-version fails if a custom style is used
Previous Next
Reported by: Kirk Kelsey <kirk.kelsey <at> 0x4b.net>
Date: Wed, 1 Sep 2010 21:09:01 UTC
Severity: normal
Found in version 23.1
Fixed in version 24.3
Done: Glenn Morris <rgm <at> gnu.org>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 6968 in the body.
You can then email your comments to 6968 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
owner <at> debbugs.gnu.org, bug-gnu-emacs <at> gnu.org
:
bug#6968
; Package
emacs
.
(Wed, 01 Sep 2010 21:09:01 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Kirk Kelsey <kirk.kelsey <at> 0x4b.net>
:
New bug report received and forwarded. Copy sent to
bug-gnu-emacs <at> gnu.org
.
(Wed, 01 Sep 2010 21:09:01 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
I've run into a problem with vc-hg-next-version and how it
identifies the maximum valid revision (the tip). Mercurial allows
for a templating of output for some commands, but
vc-hg-next-version identifies the tip using a regular expression
that only matches the "default" output style.
The patch below works by specifying the style explicitly.
Something similar is done for vc-hg-revision-table, where the
template for the log command is specified. FWIW, I took at look
at the other uses of vc-hg-command and didn't see any other cases
where the style seemed relevant. It is possible to work around
the issue by setting vc-hg-global-switches to include "--config
ui.style=default" (not using the --style option).
Thanks,
Kirk
--- /tmp/vc-hg.el 2010-09-01 16:48:56.000000000 -0400
+++ /tmp/vc-hg.fixed.el 2010-09-01 16:50:18.597813591 -0400
@@ -354,7 +354,7 @@
(let ((newrev (1+ (string-to-number rev)))
(tip-revision
(with-temp-buffer
- (vc-hg-command t 0 nil "tip")
+ (vc-hg-command t 0 nil "tip" "--style=default")
(goto-char (point-min))
(re-search-forward "^changeset:[ \t]*\\([0-9]+\\):")
(string-to-number (match-string-no-properties 1)))))
[Message part 2 (text/html, inline)]
Information forwarded
to
owner <at> debbugs.gnu.org, bug-gnu-emacs <at> gnu.org
:
bug#6968
; Package
emacs
.
(Wed, 01 Sep 2010 21:53:01 GMT)
Full text and
rfc822 format available.
Message #8 received at 6968 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Well, I missed at least one other case where this can be a
problem. In vc-hg-working-revision, vc-hg-command isn't used.
In that case, the log output may be wrong, leading to failed
version identification. The work-around doesn't apply.
[Message part 2 (text/html, inline)]
Reply sent
to
Glenn Morris <rgm <at> gnu.org>
:
You have taken responsibility.
(Sat, 01 Dec 2012 02:15:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Kirk Kelsey <kirk.kelsey <at> 0x4b.net>
:
bug acknowledged by developer.
(Sat, 01 Dec 2012 02:15:02 GMT)
Full text and
rfc822 format available.
Message #13 received at 6968-done <at> debbugs.gnu.org (full text, mbox):
Version: 24.3
Sorry for the horrendous delay; change applied to trunk. Thanks.
Kirk Kelsey wrote:
> Well, I missed at least one other case where this can be a
> problem. In vc-hg-working-revision, vc-hg-command isn't used.
Looks like this was independently changed in the meantime.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Sat, 29 Dec 2012 12:24:04 GMT)
Full text and
rfc822 format available.
This bug report was last modified 12 years and 175 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.