On Tue, 2008-06-10 at 11:36, Paul Cunningham wrote:
> Steve,
> 
> On Tue, 2008-06-10 at 10:11, Steve Christensen wrote:
> > This is a code review request for the update of SFWcvs to version 1.11.23.
> > 
> > The webrev is at
> > 
> > http://companion.sunfreeware.com/downloads/cvswebrev/
> > 
> > Details:
> > 
> > 1.  Added a METADATA file.
> > 2.  Updated version numbers and copyright dates where needed.
> > 3.  Added new source file.
> > 4.  Removed SCCS ids and .SUFFIXES: line.
> > 
> > This simple update builds cleanly in NV 87, the package installs properly, 
> > and
> > creates a working cvs system.
> 
> This looks okay to me, you may want to change the VENDOR= url to be the
> same as that in the METADATA file though.

in the src/pkgdefs/SFWcvs/pkginfo.tmpl file that is.

Paul
-- 
----------------------------------------------------------------------
Paul Cunningham
Software Engineer
Tadpole Computer Products

Reply via email to