Re: trouble with 'cvs commit -l' in visad/data/netcdf

Bill,

>Date: Fri, 13 Mar 1998 18:46 CST 
>From: "William L. Hibbard" <WHIBBARD@xxxxxxxxxxxxx>
>To: visad-list@xxxxxxxxxxxxx
>Subject: trouble with 'cvs commit -l' in visad/data/netcdf 

In the above message, you wrote:

> I just switched demedici over to jdk1.2beta3 and its Java3D.  That is,
> /opt/java and /opt/java3d now point at the new software.  This has
> caused a number of code changes, including the compile error:
>  
> > data/netcdf/Vetter.java:104: Assignment expressions are not permitted as 
> > the third subexpression of a conditional expression (that
> >                             : upperVettingLimit = fillValue - 1;
> >                                                 ^
> > 1 error
>  
> I fixed this by changing ': upperVettingLimit = fillValue - 1;' to
> ': (upperVettingLimit = fillValue - 1);'.

A better fix is to eliminate the "upperVettingLimit =" clauses in both
branches of the ternary operator (I don't know why they were there at
all).  I've done this and committed the modification.

> However, I could not check
> it in:
>  
> > Checking in Vetter.java;
> > /upc/share/visad/cvsroot/visad/data/netcdf/Vetter.java,v  <--  Vetter.java
> > new revision: 1.4; previous revision: 1.3
> > /usr/local/gnu/bin/diff: invalid option --
> > /usr/local/gnu/bin/diff: Try `/usr/local/gnu/bin/diff --help' for more 
> > information.
> > ci: /upc/share/visad/cvsroot/visad/data/netcdf/Vetter.java,v: diff failed
> > ci aborted
> > cvs server: could not check in Vetter.java
>  
> This may have something to do with cvs between domains.

It had to do with the user "visad@xxxxxxxxxxxxxxxx" not using the
correct ci(1) and diff(1) utilities.  After changing the PATH setting
for user "visad", server CVS seems to work now (I used it to commit the
above change).

Please let me know if you encounter any other problems.

--------
Steve Emmerson   <http://www.unidata.ucar.edu>