commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Rall <>
Subject CVS interface
Date Wed, 13 Mar 2002 07:57:22 GMT
[moved to infrastructure@]

"Michael A. Smith" <> writes:

> On 13 Mar 2002 wrote:
> >  Log:
>>  Removed bad line-endings (multiple ^Ms).
> [snip]
> >   Revision  Changes    Path
>>   1.5       +338 -338  jakarta-commons/collections/src/java/org/apache/commons/collections/
> I find it interesting that cvs says the entire file changes (+338 -338),
> yet if you look at the "colored diff" using viewcvs, it says "No changes"

ViewCVS ignores whitespace by default when generating diffs.  See the
view_diff() function in lib/, and the diff_format and
hr_ignore_white properties in cgi/viewcvs.conf.dist for the details.
You can take off the diff_format=h argument from the query string (or
just change the "h" to a "u") to see the unidiff in ViewCVS.

The change_summary() function of the CVSROOT/ script
(which is fired off by CVSROOT/loginfo) is not setup to ignore
white space.  Adding the -b option would turn that on.

                    open(DIFF, "-|")
                        || exec 'cvs', '-Qn', 'diff', '-u',
                              "-r$prev_rev", "-r$rev", $file;

It would present a more uniform (and arguably therefore more
user-friendly) interface if the default views for the ViewCVS and CVS
commit log diffs were normalized, but perhaps that's over simplifying
matters.  Thoughts?

- Dan

To unsubscribe, e-mail:   <>
For additional commands, e-mail: <>

View raw message