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

Re: TortoiseMerge window title (issue #345) not fixed in 1.4.4, and a suggestion

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: 2007-07-06 10:40:48 CEST

On 7/6/07, Flavio Stanchina <flavio@stanchina.net> wrote:
> Stefan Küng wrote:
> > Issue #345 is not a bug fix but a new feature. And per our rules, only
> > bugfixes get applied to the stable branches. [...]
> That's fine, it's what I would expect from a well managed project. I
> have a question, however: do you always follow Subversion releases? If
> that's the case, maybe the policy should be tweaked a bit to allow
> features like this that don't depend on new Subversion stuff to trickle
> into the stable version.

We don't follow always the Subversion releases, at least not for
bugfix releases. If we find a serious or really annoying bug in TSVN,
we release a new version even if there's no new Subversion version.
But we follow the Subversion major releases. That's simply because
those major releases include API changes in the Subversion lib, and
that's reflected in our code.

> > About your request to use the filename if no descriptive name is given:
> > I have to decline that one too. Because most of the time, we're dealing
> > with temp files in TSVN. And to use the filename of a tempfile is just
> > plain ugly and bad - because users will get confused.
> I would argue that when we're dealing with tempfiles, TMerge was called
> by another program (the TortoiseSVN context menu or any other plugin or
> IDE that chooses to use TMerge -- for example we're using the PushOk
> plugin here) and that should have taken care of setting th descriptive
> name too. If the descriptive name is *not* set, we're most likely
> dealing with real files: I use TMerge all the time to diff files because
> I like its interface.

That's a good point! I think we really could use the filename in those cases.

> Anyway, I certainly prefer to see the name of a tempfile rather than
> nothing... It's not like your standard command-line diff will hide file
> names if no labels are supplied.

I'll make the change maybe this evening.


  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 Jul 6 10:40:31 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.