[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Enhancement for the issue tracker integration

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: 2007-10-12 23:06:50 CEST

Mark Phippard wrote:
> On 10/12/07, Stefan Küng <tortoisesvn@gmail.com> wrote:
>> Subversion 1.5 has the ability to set revision properties with every
>> commit (and other commands which create a new revision). I thought we
>> could use this new feature to add a revision property for those who want
>> it. Of course, it is also possible to get that revision property when
>> fetching the log information with SVN 1.5, so it will still be possible
>> to show the associated bug ID with every log message.
>> I've already written a proposal on how this could be done:
>> http://issues.tortoisesvn.net/index.php?do=details&task_id=405
>> Please comment on this now so we can get the specs right and can
>> implement them before SVN 1.5 gets released.
> Do we still think this is worth doing? Way back when, the reason I
> thought this would be good is because I imagined being able to ask
> Subversion questions like "which revisions reference issue 123?"
> There is still no way to do that ... and probably won't be.

You're right, there's no way to answer the question "which revisions
reference issue 123". But issuetrackers which have direct access to the
repository can modifiy or read revision properties much easier than log
messages (no need to parse them).

> On the plus side, it makes things easier for people writing hook
> scripts that want this information.

And don't forget: the log message itself won't have to be modified (it
still can be, of course).

> Having used the feature now for so long, the log messages feels like
> it is the right solution. Especially since command line users and
> clients that do not implement the feature can so easily participate.
> I think when you introduced the regex stuff, it really solidified that
> the log message was the right way to do this.
> I think we might be doing users a favor by not supporting an option like this.

How about not making it an option but simply always set that property?
That way, there's no need for additional configuration for the user, but
the revision property will always be there and can be used by those who
need it.
It shouldn't be too much work to implement this: we already know the
issue IDs from the log message the user entered, and adding the revision
property to the ctx structure before commits should be easy enough.


   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Fri Oct 12 23:27:04 2007

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.