Venky, This looks okay to me except for the SUNW package dependency stuff, but I may be misunderstanding what you are doing with it ....
SUNWprivoxyr has a 'depend' file but SUNWprivoxyu doesn't. SUNWprivoxyr depends on SUNWprivoxyu (by the way the comment is wrong there) ? And there is also a dependency on SUNWpcre. Shouldn't SUNWprivoxyr just have the default 'depend'. And SUNWprivoxyu depend on SUNWprivoxyr & SUNWpcre (and default stuff)? Paul Venky wrote: > Have uploaded the updated webrev incorporating the comments here: > http://cr.opensolaris.org/~venkytv/privoxy-20080819/ > > The webrev does not include the source tarball since webrev generates > a bunch of errors when it encounters binary files. It has been > checked into the workspace though. > >> Do you need to run 'autoconf', could you have just >> used the 'configure' that came with the src? > > I checked this again. The source does not include the configure > script. -- ---------------------------------------------------------------------- Paul Cunningham Software Engineer Tadpole Computer Products
