Bug 100853

Summary: Doesn't handle errors from pre-commit checks (commitinfo)
Product: [Applications] cervisia Reporter: ieure
Component: generalAssignee: Christian Loose <christian.loose>
Status: RESOLVED DUPLICATE    
Severity: normal    
Priority: NOR    
Version First Reported In: 2.2.2   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description ieure 2005-03-05 00:01:55 UTC
Version:           2.2.2 (using KDE 3.3.2,  (3.1))
Compiler:          gcc version 3.3.5 (Debian 1:3.3.5-8)
OS:                Linux (i686) release 2.6.10-1-686

I have a commitinfo script that runs my code through a syntax checker, and refuses to commit broken code.

When I commit a file which fails the check, Cervisia incorrectly updates it's status of that file to 'Up-to-date,' when it should remain 'Locally Modified.'

From the log window:

-- snip --
cvs -f commit -l -m 'Debug.' 'test.php' 2>&1 
Parse error: parse error in /tmp/cvs-serv30395/test.php on line 202
Errors parsing test.php
cvs commit: Pre-commit check failed
cvs [commit aborted]: correct above errors first!
[Exited with status 1]
-- snip --

This is frustrating when unmodified files are hidden, since I have to unhide and hunt through the list for the file I tried to commit, or update to get the correct status back.
Comment 1 Christian Loose 2005-05-17 12:09:21 UTC

*** This bug has been marked as a duplicate of 79616 ***