Vivek Titarmare wrote: > Hi, > > As mentioned below almost all the points are updated and the new webrev is > uploaded. The url is http://cr.opensolaris.org/~vivekrt/antlr277.
As as been mentioned multiple times. You need to clean up the CDDL headers in all files to match this: http://cr.opensolaris.org/~vivekrt/antlr277/usr/src/pkgdefs/Makefile.html Are you planning to have multiple versions of antlr in OpenSolaris at the same time? Why? If you don't need multiple versions then I would get rid of the 277 suffix. antlr277.3 If a user types: $ man antlr What do they get? What about "$ man antlr227" ? Cheers, Jim -- Jim Walker, http://blogs.sun.com/jwalker Sun Microsystems, Broomfield, Colorado
