#47095 `cvs diff' discards error information

Package:
cvs
Source:
cvs
Description:
Concurrent Versions System
Submitter:
Date:
2011-12-04 13:48:21 UTC
Severity:
wishlist
#47095#5
Date:
1999-10-10 10:54:05 UTC
From:
To:
Package: cvs
Version: 1.9.29-1
Severity: wishlist

This makes it rather hard to write reliable scripts that invoke `cvs
diff'.

GNU diff solves this problem by exiting 0 for no differences, 1 for
some differences and 2 for error.  It would be best if CVS could do
the same.

ttfn/rjk

#47095#16
Date:
2007-10-05 11:20:32 UTC
From:
To:
Dear CVS user,

Thanks for your interest in CVS and the bug report you have contributed [1].

Debian's cvs package has ~120 old bugs, most of them are couple of years old.

As part of a bug triage I'm doing for several packages, I would like your help
with verifying your bug is still relevant or getting your approval for closing
it.

The current cvs version in Debian is 1.12.13-8 (shared by stable, testing and
unstable).

Feel free to contact me for questions or if you need help.

[1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=47095

Thanks.

#47095#19
Date:
2007-10-05 11:20:32 UTC
From:
To:
Dear CVS user,

Thanks for your interest in CVS and the bug report you have contributed [1].

Debian's cvs package has ~120 old bugs, most of them are couple of years old.

As part of a bug triage I'm doing for several packages, I would like your help
with verifying your bug is still relevant or getting your approval for closing
it.

The current cvs version in Debian is 1.12.13-8 (shared by stable, testing and
unstable).

Feel free to contact me for questions or if you need help.

[1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=47095

Thanks.

#47095#24
Date:
2007-10-05 14:01:47 UTC
From:
To:
Lior Kaplan writes:

The bug still exists.

ttfn/rjk

#47095#33
Date:
2011-12-04 13:35:43 UTC
From:
To:
Hi,

I just looked at this, and it is totally nontrivial, as about
the entire code can exit(1) on error. Our best chance seems to
be to always exit(2) on error for _all_ subcommands (and glo-
bally) and then return 1 on diff differences.

I’ll accept patches ☺

bye,
//mirabilos