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

Re: [TSVN] Re: Integration with bug tracking tools

From: SteveKing <steveking_at_gmx.ch>
Date: 2004-08-22 16:34:10 CEST

Mark Phippard wrote:

> No offense, but that is just BS. I am a developer too, and that is also
> one of my favorite lines to give to users when I do not want to do
> something. This feature is not adding complexity. First off, none of

Don't know what "BS" means. But it sounds like something not really
nice. Is that really where this discussion is going?

> these feature are exposed in the UI as settings. They are properties that
> have to be set on folders in the repository. Someone has to explicitly
> want and enable this feature. A hapless user does not have to learn and

"someone" has to want and enable it. But what about all the other
developers?

> understand these settings on a preference page. Second, the code to
> implement this feature would be vritually identical to code already added
> to verify if the issue ID is numeric. This code should actually be
> simpler. So let's retire the bloated/complicated code argument shall we?

I've never said it couldn't be done or that it's complicated or much
work to do it.

> So then you have decided that someone who needs this feature is "out of
> control". Nevermind the fact that this feature causes no harm to the
> people that do not want it and do not turn it on. But for someone that

But it causes harm to those who don't want it, but are _forced_ to use
it because someone else turned it on.

> needs it, you would rather have a system where good intentioned users can
> forget to fill out the bug ID field, forcing the admin to discover this
> fact later, and then manually rewrite the log message in the correct
> format from the command line? Doesn't Fogcreek have any customers in

Even _if_ this feature would be implemented in TSVN, there are ways
around it:
- just use another client
- remove the properties from the working copy folders, commit the
changes in the files and set the properties back
- enter a dummy issue number

So all this feature would really do is driving people away from
TortoiseSVN to other Subversion clients.

> regulated indiustries like Health Care or Finance? I suspect a large
> percentage of our customers would have a feature requirement like this.
> However, as I have said, I think the number one reason is to make the
> feature more usable for those people that want the preponderance of their
> work to have an issue ID associated. If they do not care if every commit
> has an issue ID, then they should not care if they user enters a '0' or
> some other value like 'N/A'. But again, they could also not go out of
> their way to turn this feature on.

But if those people _really_ care about that, they must do that in a
pre-commit-hook on the repository. Clients _can't_ assure that this
"policy" is kept.

> I agree with your first argument, I just do not think it applies here.
> This is a question with no right answer. Why does Tortoise allow the user
> to specify their own diff/merge tools? Why does Tortoise allow the user

That feature was added because in the beginning, TortoiseSVN didn't have
its own diff/merge tool.

> to control where decorators are used and how they behave? Answer, because
> there is no single answer. I have already given several reasons why I
> think someone would legitimately want this feature, all you have said is
> that you would not want it yourself. Neither would I. I think that point
> is irrelevant.

And We have given you several reasons why this can't be done in a
client. You want to make _sure_ that an issue number is assigned to a
commit. A client can't do that - there are many ways around it. If you
want to ensure that, you must use a pre-commit-hook.
Adding this to Subversion clients will just make the life of devs harder
and more complicated.

Just write a pre-commit-hook for your bug tracking tool and you're done.
Then you can provide your customers with a solution that works, with no
workarounds. Sure - it means that you have to write that script
yourself, but I think that's the only drawback on this.

Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Sun Aug 22 17:43:41 2004

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.