Just of couple of comments ...

1. src/cmd/AfterStep/install-sfw
    This still has the extra double-spaces in the CDDL header

2. src/cmd/AfterStep/METADATA
    The SRC: url isn't the same version as what you are using.
    Also out of interest the www.afterstep.org webite says the
    latest version is ..
     "Latest Version 2.2.9 was released on July 8, 2009"
    why are you not using that?

Paul

Steve Christensen wrote:
> This is a code review request for the update of the AfterStep package to 
> version 1.8.11.  This will be used in the CCD for the next Solaris 10 
> release.  The webrev is at
> 
> http://companion.sunfreeware.com/downloads/afterstepwebrev/
> 
> Details:
> 
> 1. Updated source file.
> 2. Updated copyright dates and form and version numbers where needed.
> 3. Added METADATA file.
> 4. SCCS ident lines removed.
> 5. CDDL extra spaces removed.
> 
> This builds properly and creates a working afterstep.
> 
> Steve Christensen

-- 
Paul Cunningham
Software Engineer

Reply via email to