From: Dmitry Gutov <dgutov@yandex.ru>
To: Juri Linkov <juri@jurta.org>
Cc: Sam Steingold <sds@gnu.org>, emacs-devel@gnu.org
Subject: Re: bug-reference and revisions
Date: Fri, 28 Dec 2012 03:20:47 +0400 [thread overview]
Message-ID: <50DCD7CF.5090307@yandex.ru> (raw)
In-Reply-To: <87han7cdl6.fsf_-_@mail.jurta.org>
On 28.12.2012 2:45, Juri Linkov wrote:
>>>>> - from 109911 (see the new test).
>>>>> + from Bug#109911 (see the new test).
>>>>
>>>> This change is inaccurate. 109911 is a Bazaar revision, not a bug number.
>>>
>>> To avoid ambiguity and misunderstanding revisions should have a prefix
>>> similar to bug numbers, e.g. revno#109911 or revno:109911.
>>
>> Thanks, will do next time.
>> I guess I'm used to Git, where you don't usually prefix commit ids with
>> anything.
>
> Support for links to revision numbers could be added to bug-reference.el
> to fontify strings like "revno:109911" where clicking on them will open
> http://bzr.savannah.gnu.org/lh/emacs/trunk/revision/109911
That would be nice.
> This is possible with a small patch that adds these formats:
>
> ...
>
> Then evaluating this will open the link:
> (setq bug-reference-url-format
> "http://bzr.savannah.gnu.org/lh/emacs/trunk/revision/%s")
>
> but unfortunately it will override "http://debbugs.gnu.org/%s".
>
> To handle two URL types requires customizing `bug-reference-url-format'
> to a function that would decide which URL format to use.
Why not add a function that would do exactly that and then set
bug-reference-url-format to it in .dir-locals.el?
next prev parent reply other threads:[~2012-12-27 23:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <E1ToH53-0003nX-W3@vcs.savannah.gnu.org>
2012-12-27 18:24 ` /srv/bzr/emacs/trunk r111349: * lisp/progmodes/cperl-mode.el (cperl-calculate-indent): Do not stagger Dmitry Gutov
2012-12-27 19:39 ` Juri Linkov
2012-12-27 20:12 ` Sam Steingold
2012-12-27 21:55 ` Dmitry Gutov
2012-12-27 22:45 ` bug-reference and revisions Juri Linkov
2012-12-27 23:20 ` Dmitry Gutov [this message]
2012-12-28 0:49 ` /srv/bzr/emacs/trunk r111349: * lisp/progmodes/cperl-mode.el (cperl-calculate-indent): Do not stagger Stephen J. Turnbull
2012-12-28 4:36 ` Xue Fuqiao
2012-12-28 6:49 ` Stephen J. Turnbull
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=50DCD7CF.5090307@yandex.ru \
--to=dgutov@yandex.ru \
--cc=emacs-devel@gnu.org \
--cc=juri@jurta.org \
--cc=sds@gnu.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).