Bug 100853 - Doesn't handle errors from pre-commit checks (commitinfo)
Summary: Doesn't handle errors from pre-commit checks (commitinfo)
Status: RESOLVED DUPLICATE of bug 79616
Alias: None
Product: cervisia
Classification: Applications
Component: general (other bugs)
Version First Reported In: 2.2.2
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Christian Loose
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-03-05 00:01 UTC by ieure
Modified: 2005-05-17 12:09 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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 ***