Looks okay to me, but in usr/src/lib/antlr2/Makefile.sfw why not use
$(CONFIGURE_OPTIONS), from Makefile.master, for the configure, eg ...
change ...
48 ./configure --prefix=$(CFGPREFIX) )
to ...
./configure $(CONFIGURE_OPTIONS)
Paul
Vivek Titarmare wrote:
>
> I have posted a webrev for "antlr 2.7.7" package which has a bug fix for
> 6825758 and would like to request a code review.
>
> Please see http://cr.opensolaris.org/~vivekrt/6825758-antlr2bug/ and provide
> any comments as needed if there are any issues which I need to correct.
--
----------------------------------------------------------------------
Paul Cunningham
Software Engineer
Tadpole Business Unit
