On 23/08/07, Ron Wilson <ronw.mrmx@gmail.com> wrote:
> On 8/23/07, Simon Large <simon.tortoisesvn@googlemail.com> wrote:
> > On 23/08/07, Matuschka, Sebastian <Sebastian.Matuschka@gcd-solutions.de> wrote:
> > > i've made a pre-commit program for subversion that checks for empty log
> > > messages and realized that if i copy a directory by clicking on Branch /
> > > Tag in the context menu and enter no log message that the log message
> > > will be "made a copy". So my pre-commit thougt the user entered a
> > > message. At the moment i catch not only empty log messages but also
> > > "made a copy". I've looked in the code of Tortoise an found the line
> > > where "made a copy" is sent as log message when the log message is empty
> > > in SVN.cpp:
> > > if (logmsg.IsEmpty())
> > > m_pctx->log_msg_baton2 =
> > > logMessage(CUnicodeUtils::GetUTF8(CString(_T("made a copy"))));
> > > As far as i know this is the only exception, but i don't know it
> > > exactly, i'm not familiar with the tortoise code.
> > > So my questions are:
> > > Why is this there but not when other operations are made?
> > > Are there any similar cases which i have to catch as empty log message?
> > >
> > > I looked in the newest SVN.cpp in the Version-Control-System, but it's
> > > still there.
> >
> > There's a conflict in requirements here. Particularly in repo browser,
> > these messages are useful as it is otherwise far too easy to just
> > click through the OK button and create a copy or new folder with no
> > log message at all. Been there, done that.
> >
> > On the oher hand, having a default message defeats pre-commit hooks
> > and the tsvn:MinLogMinMessageMinLength (I know there's a Min in there
> > somewhere).
> >
> > I don't know if it's possible to do this in repo browser too, but
> > maybe if there is a min log message length set we could suppress using
> > the default message?
> >
> Seems to me that "made a copy" would likely be shorter than the
> setting of "tsvn:logminsize", anyway. Of course, "tsvn:logminsize"
> only applies to operations performed through TSVN.
>
Yes, but the "made a copy" message only applies to operations
performed through TSVN too :)
--
___
oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
\ \_/_\_/> The coolest Interface to (Sub)Version Control
/_/ \_\ http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: users-help@tortoisesvn.tigris.org
Received on Thu Aug 23 17:57:59 2007