[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: Flavio Stanchina <flavio_at_stanchina.net>
Date: 2007-07-06 10:31:01 CEST

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.

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

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.

-- 
Ciao, Flavio
---------------------------------------------------------------------
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:31:03 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.