[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [E-devel] .cvsignore rampage.



Justin Patrin wrote:
> 
> 
> On 2/24/06, Kevin Brosius <cobra@compuserve.com> wrote:
> > David Seikel wrote:
> > >
> > >
> > > I will probably go on a cvs rampage this weekend.  I'll start with
> > > getting the .cvsignore files in order, as they are in need of some tlc,
> > > and I might end up with getting engage and enotes up to date with the
> > > recent EWL API change if their authors don't beat me to it.  Somewhere
> > > in the middle I am likely to apply indent to whatever I'm considered to
> > > be responsible for.
> > >
> > > You have plenty of time to object.  B-)
> >
> > Just please... please... please...
> >
> > Don't commit indent runs and code changes in the same cvs transaction.
> > It makes it impossible to tell what changed.
> >
> 
> s/impossible/hard/
> 
> And you can always diff -b (--ignore-space-change).
> 

Yeah, well, web webcvs supports that option, I'll buy that argument.


> I agree, though, formatting changes and real edits should not be in
> the same commit.
> 
> --
> Justin Patrin

-- 
Kevin