[Bug 56782] CVS operations give errors

[Amilcar do Carmo Lucas]

The first option is much better. But it's harder. If you provide a patch 
I'll commit ;)

Amilcar



Mario Scalas wrote:

>>------- Additional Comments From a.lucas at tu-bs.de  2003-04-03 22:52 -------
>>The combined patches from Mario and Jan, plus some changes I maid seam to
>>fix this bug.
>>    
>>
>
>Hope so :)
>Anyway playing around with the CVS part I discovered this behaviour (at least 
>my cvs program does so ;): when I commit without a message (leave the message 
>in the dialog blank), cvs comply about  some option to be chosen by the user. 
>The cvspart ignore the question and consider commit as failed but it is not 
>what I expect ;)
>I believe that a message should be posted anyway (as good programming 
>practice): we have two choices that come in mind at once (well, I could be 
>missing something ... ;)
>(1) Modify the commit dialog to ensure that the user put some text, limiting 
>his/her freedom but enforcing "good manners in using the repository".
>(2) Permitting to leave the message blank but making cvs to add a default 
>comment such as a fantastic:
>
>...
>    QString message = d.logMessage();
>    if (!message.isEmpty())
>        message = " -m " + KShellProcess::quote(message);
>    else
>	message = " -m " + KShellProcess::quote("<No comment for this commit>");
>...
>
>What do you think?
>
>Regards,
>Mario.
>
>
>_______________________________________________
>Kdevelop-devel mailing list
>Kdevelop-devel at barney.cs.uni-potsdam.de
>http://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
>  
>

-- 
Amilcar Lucas

---------------------------------------------------------------------------
             Institut fuer Datentechnik und Kommunikationsnetze
               Technische Universitaet Braunschweig, Germany      

Amilcar do Carmo Lucas                  Tel. : +49 531 391 3751
Raum 1315                               Fax  : +49 531 391 3750
Hans-Sommer Str. 66                     Email: amilcar at ida.ing.tu-bs.de
D-38106 Braunschweig
           http://www.ida.ing.tu-bs.de/people/amilcar/home.e.shtml
---------------------------------------------------------------------------


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20030404/ae12f024/attachment.html>